relax legacy solo payout verification in tProxy#589
Merged
plebhash merged 1 commit intoJul 2, 2026
Conversation
Member
Author
|
Tested against PyBlock Pool endpoint (https://pyblock.xyz:8443/sv2.php) and it's working fine now. |
plebhash
approved these changes
Jul 1, 2026
plebhash
reviewed
Jul 1, 2026
GitGab19
force-pushed
the
improve-payout-verification-in-tproxy
branch
3 times, most recently
from
July 2, 2026 08:15
4c9b5c9 to
10988a4
Compare
plebhash
force-pushed
the
improve-payout-verification-in-tproxy
branch
from
July 2, 2026 13:49
10988a4 to
e836c91
Compare
plebhash
approved these changes
Jul 2, 2026
- 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
force-pushed
the
improve-payout-verification-in-tproxy
branch
from
July 2, 2026 14:32
e836c91 to
03cb0a1
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.