Skip to content

Ignore duplicate stratum jobId#1731

Open
mutatrum wants to merge 2 commits into
bitaxeorg:masterfrom
mutatrum:handle-duplicate-mining-notify
Open

Ignore duplicate stratum jobId#1731
mutatrum wants to merge 2 commits into
bitaxeorg:masterfrom
mutatrum:handle-duplicate-mining-notify

Conversation

@mutatrum

@mutatrum mutatrum commented May 29, 2026

Copy link
Copy Markdown
Collaborator

In some cases, a pool can send work that's already on the ASIC. These messages should be ignored.

Also reset extranonce_2 on mining.set_extranonce

Fixes #1587
Replaces #1694

@github-actions

github-actions Bot commented May 29, 2026

Copy link
Copy Markdown

Test Results

  2 files  ±0    2 suites  ±0   0s ⏱️ ±0s
103 tests ±0  103 ✅ ±0  0 💤 ±0  0 ❌ ±0 
105 runs  ±0  105 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b83aa3e. ± Comparison against base commit c1915b0.

♻️ This comment has been updated with latest results.

mutatrum added 2 commits June 20, 2026 12:16
Also reset `extranonce_2` on `mining.set_extranonce`
@mutatrum mutatrum force-pushed the handle-duplicate-mining-notify branch from 471fa6d to b83aa3e Compare June 20, 2026 11:16
@mutatrum mutatrum changed the title Ignore duplicate mining.notify stratum messages Ignore duplicate stratum jobId Jun 20, 2026
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.

esp-miner is unable to handle difficulty change following with "mining.notify" when cleanJob set to false

1 participant