Skip to content

signer: Refresh on nonces >= 500 - #4028

Open
j0sh wants to merge 2 commits into
masterfrom
ja/signer-nonces
Open

signer: Refresh on nonces >= 500#4028
j0sh wants to merge 2 commits into
masterfrom
ja/signer-nonces

Conversation

@j0sh

@j0sh j0sh commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator

The orchestrator caps nonces at 600 so require a refresh at 500 in case the payment generates 100 tickets (the maximum).

The orchestrator caps nonces at 600 so require a refresh at 500 in
case the payment generates 100 tickets (the maximum).
@codecov

codecov Bot commented Aug 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 35.58179%. Comparing base (90ff849) to head (9629ce8).

Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                 @@
##              master       #4028         +/-   ##
===================================================
+ Coverage   35.54150%   35.58179%   +0.04029%     
===================================================
  Files            174         174                 
  Lines          45226       45231          +5     
===================================================
+ Hits           16074       16094         +20     
+ Misses         27879       27863         -16     
- Partials        1273        1274          +1     
Files with missing lines Coverage Δ
server/remote_signer.go 67.40891% <100.00000%> (+0.33325%) ⬆️

... and 5 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 90ff849...9629ce8. Read the comment docs.

Files with missing lines Coverage Δ
server/remote_signer.go 67.40891% <100.00000%> (+0.33325%) ⬆️

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant