search

stratum-mining / stratum

stratum
https://stratumprotocol.org
Other
192 stars 118 forks source link

refactor: Error handling #778

Open plebhash opened 4 months ago

plebhash commented 4 months ago

SRI error handling can be improved in a few different ways, this is an umbrella issue tracker

the issues below will require some substantial refactoring effort, so they are not necessarily trivial

roles

protocols

pavlenex commented 2 months ago

@plebhash How about things mentioned in https://github.com/stratum-mining/stratum/issues/50? Should we close that one both issues sound similar but context seems a bit different, and this issue is a bit more concrete in defining what needs to be done?

plebhash commented 2 months ago

@plebhash How about things mentioned in #50? Should we close that one both issues sound similar but context seems a bit different, and this issue is a bit more concrete in defining what needs to be done?

778 was originally written to address Error handling on roles, while #50 is referring to Error handling on low level crates (protocols), so they were originally slightly different things

but I'm editing the description of #778 to include a reference to #52 (which was the only sub-issue that was still open on #50)

and I'm closing #50 so we use this as a tracker instead