stratum-mining / stratum

stratum
https://stratumprotocol.org
Other
204 stars 121 forks source link

redundant fields in DeclareMiningJob message #625

Open 4ss0 opened 11 months ago

4ss0 commented 11 months ago

We receive all the outputs from the AllocateMiningJobToken.Success message from the JD server, so the fields: coinbase_tx_value_remaining and tx_hash_list_hash in DeclareMiningJob in JD client seems to be redundant at this point of the flow.

GitGab19 commented 4 months ago

@Fi3 I see that coinbase_tx_value_remaining is not present anymore on specs (DeclareMiningJob). What do you think about the comment on tx_hash_list_hash?