This reverts commit 59b00c1.

…handler"

This reverts commit 002e30b.

..to `TryFrom<(u8, &'a mut [u8])> for PoolMessages`

the GetSize implementation is not accounting for self.version.get_size()!

that is causing NoiseEncoder to allocate a buffer that's smaller than what is actually needed, which eventually triggers FramingSv2Error(BinarySv2Error(WriteError)) during encoding

1. Removed `NoiseHeader` struct in favor of three constants defined at
the top of the file.
2. Added documentation and changed visibility to `pub(crate)` where
needed.
3. Removed `Header::Default` and `Sv2Frame::Default` impls as they are
unused.
4. Removed `unwrap()`s

The `Frame` trait is used solely by a single struct and it is only
adding biolerplate to the code without benifits. We could consider
re-adding it in the future if needed.

..followed by their `impl`

- Also removed double `impl HandShakeFram {}` occurance

Its currently producing too much false alarms which
can block new pull requests as well as make it harder to find
issues in new pull requests.

this is a paliative solution to #1028

we make MG CI slightly less worse by breaking tests into multiple jobs and allowing us to re-run false alarms in a more agile way

MG Tests are also separated from Tarpaulin

Github only allows branch protection rules based on specific CI jobs. So we are adding a new job that just takes all the other MG test jobs as a prerequisite, and we use this as an umbrella rule for branch protection.

With the goal to write more tests, this commit moves JDC initialisation
logic from `main.rs` to `lib.rs` so its easier to test and also separate
the actual `lib` code from the binary.

Co-authored-by: plebhash <147345153+plebhash@users.noreply.github.com>

Add parallelism to cpu miner

doc: Add requirements on interoperability tests

Changes:
- Add listener on exit signals
- Add channel to unbind listener as well
- Organize main fn to handle signals and ownership of sockets
- Fix typos in docs
- Format imports
- Remove unused spawn
- Simplifies the code a bit reducing nesting and matches with unused
  arms

(cherry picked from commit f91be81)

Co-authored-by: jbesraa <jbesraa@gmail.com>

Graceful shutdown for mining-proxy

this is the actual fix for #912

use handle_result! macro

following suggestion by @lorbax #1025 (comment)

the implementation diverged a bit from the suggestion, but it was still a good reminder that we should leverage `handle_result!` macro here

avoid JDS dropping connection on `SubmitSolution` with missing txs

`test_diff_management`: adjust `total_run_time` to  60s

before issue #853 was reported, JDS would simply support async jobs by default and completely ignore this flag.

but checking this flag implies that JDS could either support async jobs or not, and that is what this commit does.

a new `asyn_mining_allowed` parameter is introduced to the TOML config files, and that is used when:
- checking for the flags of `SetupConnection` messages
- responding to `AllocateMiningJobToken` messages

as already described in the previous commit, we introduced a new JDS config parameter (`async_mining_allowed`)

so now, we need 2 separate tests:
- a JDS that supports async jobs
- a JDS that does not support async jobs

and for each test, we need a mock that:
- sends a `SetupConnection` with flag 0 and asserts the expected outcome
- sends a `SetupConnection` with flag 1 and asserts the expected outcome

if JDS does not support async jobs and receives a `SetupConnection` with flag 1, the expected outcome is `SetupConnection.Error`
in all other cases, the expected outcome is a `SetupConnection.Success` with the same flag as the original `SetupConnection`

Enhance SetupConnection Handling and Implement Job declaration Flag Checks

…SCRIMINANT to SV2_JOB_DECLARATION_PROTOCOL_DISCRIMINANT

…ocol

Remove job distribution protocol

`Upstream::parse_incoming` only calls `handle_message_mining`, which is not able to handle a `SetupConnection.Success` (because it belongs to the "Common Messages" category, not "Mining Messages")

…ction

JDC parse incoming mining messages from upstream after SetupConnection

make `no_std` into an optional feature on `protocols` crates

required to unblock #985 (comment)

add missing `impl` for `Seq0255` on `binary_sv2/serde-sv2`

requirement to unblock #985 (comment)

requirement to unblock #985 (comment)

import `TryInto` on `template_distribution_sv2`

- Removed dependencies related to TOML configuration.
- Updated configuration handling to use ext-config crate instead.
- Refactored code to eliminate TOML-specific logic.

- Removed dependencies related to TOML configuration.
- Updated configuration handling to use ext-config crate instead.
- Refactored code to eliminate TOML-specific logic.

- Removed dependencies related to TOML configuration.
- Updated configuration handling to use ext-config crate instead.
- Refactored code to eliminate TOML-specific logic.

- Removed dependencies related to TOML configuration.
- Updated configuration handling to use ext-config crate instead.
- Refactored code to eliminate TOML-specific logic.

- Removed dependencies related to TOML configuration.
- Updated configuration handling to use ext-config crate instead.
- Refactored code to eliminate TOML-specific logic.