Skip to content

enable payout verification in tProxy in case of SOLO mining - #497

Merged
GitGab19 merged 4 commits into
stratum-mining:mainfrom
GitGab19:solo-mining-in-tproxy
May 15, 2026
Merged

enable payout verification in tProxy in case of SOLO mining#497
GitGab19 merged 4 commits into
stratum-mining:mainfrom
GitGab19:solo-mining-in-tproxy

Conversation

@GitGab19

Copy link
Copy Markdown
Member

Closes #308

@GitGab19
GitGab19 force-pushed the solo-mining-in-tproxy branch from 8962682 to f3b29cf Compare May 13, 2026 17:43
Comment thread stratum-apps/src/payout.rs Outdated
Comment thread miner-apps/translator/src/lib/config.rs
@GitGab19
GitGab19 force-pushed the solo-mining-in-tproxy branch 3 times, most recently from 8f990cd to 9aa4148 Compare May 14, 2026 12:44
Comment thread stratum-apps/src/payout.rs Outdated
Comment thread stratum-apps/src/payout.rs Outdated
@plebhash
plebhash self-requested a review May 14, 2026 18:06
@GitGab19
GitGab19 force-pushed the solo-mining-in-tproxy branch from 9aa4148 to 9ce8712 Compare May 14, 2026 18:24

@plebhash plebhash left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did a quick overview of the 4 commits, overall good patterns and clever usage of MockUpstream to assert tProxy falls back under some scammy solo pool

no findings from gpt-5.3-codex

@plebhash

plebhash commented May 15, 2026

Copy link
Copy Markdown
Member

conflict on integration-tests/Cargo.toml is due to version bumps introduced via #495

see #487 (comment)

@Shourya742 Shourya742 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@GitGab19
GitGab19 force-pushed the solo-mining-in-tproxy branch from 9ce8712 to dcaa401 Compare May 15, 2026 07:45
@GitGab19
GitGab19 force-pushed the solo-mining-in-tproxy branch from 9118ee3 to dcaa401 Compare May 15, 2026 09:05
@GitGab19
GitGab19 merged commit ccdef6d into stratum-mining:main May 15, 2026
18 of 22 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.

tProxy should have a SOLO mining mode and check for Bitcoin address presence in the coinbase transaction

3 participants