Skip to content

relax legacy solo payout verification in tProxy#589

Merged
plebhash merged 1 commit into
stratum-mining:mainfrom
GitGab19:improve-payout-verification-in-tproxy
Jul 2, 2026
Merged

relax legacy solo payout verification in tProxy#589
plebhash merged 1 commit into
stratum-mining:mainfrom
GitGab19:improve-payout-verification-in-tproxy

Conversation

@GitGab19

@GitGab19 GitGab19 commented Jul 1, 2026

Copy link
Copy Markdown
Member

This PR relaxes the payout verification (optionally) done in tProxy when using legacy solo user_identities, in the shape of <payout_address>[.worker].

In that case, the check is about making sure jobs pay at least 90% of spendable coinbase outputs to the miner address (allowing up to 10% service fee in legacy solo mode).

This is needed to unblock stratum-mining/sv2-ui#203 and to handle other similar cases which might pop up in the future.

@GitGab19

GitGab19 commented Jul 1, 2026

Copy link
Copy Markdown
Member Author

Tested against PyBlock Pool endpoint (https://pyblock.xyz:8443/sv2.php) and it's working fine now.

Comment thread stratum-apps/src/payout.rs Outdated
@GitGab19
GitGab19 force-pushed the improve-payout-verification-in-tproxy branch 3 times, most recently from 4c9b5c9 to 10988a4 Compare July 2, 2026 08:15
@plebhash
plebhash force-pushed the improve-payout-verification-in-tproxy branch from 10988a4 to e836c91 Compare July 2, 2026 13:49
- require legacy <payout_address>[.worker] jobs to include at least 90%
  of the full reward in the miner output
- keep `sri/solo` strict, keep `sri/donate` exact
- cover the fee-output case in tests/docs
@plebhash
plebhash force-pushed the improve-payout-verification-in-tproxy branch from e836c91 to 03cb0a1 Compare July 2, 2026 14:32
@plebhash
plebhash merged commit 453e853 into stratum-mining:main Jul 2, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants