mining-proxy needs graceful death instead of panic

[bcral]

The mining-proxy seems to be panicking on closure. This makes test coverage data impossible to retrieve with llvm-cov.

This same issue used to exist on the pool and TProxy until it was fixed by #432

[bcral]

Please tag as "good first issue"!

[Priceless-P]

@bcral i’d like to work on this.

[Priceless-P]

@bcral Just to be sure, this is the file that needs to be worked on for this issue right?

[bcral]

@Priceless-P Yes, that's correct. You may want to check with @darricksee first - he may have already started on this.

[darricksee]

I haven't and probably won't get to it soon.

[darricksee]

Just a heads-up @Priceless-P - to implement this the status-channel also needs to be added to the mining-proxy before getting to this issue. You can certainly add both with this issue - take a look at how the status-channel is used in the pool role. It's basically a mechanism for the application to signal the main thread to terminate if one of the tasks has an error.

[Priceless-P]

Okay. Thank you for letting me know. @darricksee

[SumantxD]

@bcral i would like to work on this issue, can you assign it to me.

[Priceless-P]

Hi @SumantxD, I am currently working on it.

[SumantxD]

@Priceless-P ok fine

[mihir1739]

Hey is anyone working on this issue ? If not please assign this to me .

[pavlenex]

Hey @mihir1739 it seems @Priceless-P is working on it here https://github.com/stratum-mining/stratum/pull/518

[plebhash]

@Priceless-P what is the status on this? why was #518 closed without being merged?

[Priceless-P]

@plebhash Yes.

[johnnyasantoss]

Is this up for grabs? @Priceless-P