Skip to content

bugfix: key-rotation + reply SURBs #5876

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
Jul 4, 2025

Conversation

jstuczyn
Copy link
Contributor

@jstuczyn jstuczyn commented Jun 30, 2025

this PR should hopefully, probably, maybe, I think resolve the issue of suddenly running out of usable reply SURBs when a new key rotation occurs.


This change is Reviewable

Copy link

vercel bot commented Jun 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nym-explorer-v2 🛑 Canceled (Inspect) Jul 4, 2025 3:06pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
docs-nextra ⬜️ Ignored (Inspect) Visit Preview Jul 4, 2025 3:06pm
nym-next-explorer ⬜️ Ignored (Inspect) Visit Preview Jul 4, 2025 3:06pm

@jstuczyn jstuczyn force-pushed the bugfix/surb-refresh branch from f59c318 to cb8283c Compare June 30, 2025 17:32
@jstuczyn jstuczyn mentioned this pull request Jun 30, 2025
@pronebird
Copy link
Contributor

It would be better to replace log with tracing or run clippy in a separate PR to avoid such a massive bloat.

jstuczyn added 3 commits July 2, 2025 11:42
includes, but is not limited to: unconditionally reseting sender tag on restart, limiting number of surb re-requests, resetting stale surbs on load
@jstuczyn
Copy link
Contributor Author

jstuczyn commented Jul 2, 2025

It would be better to replace log with tracing or run clippy in a separate PR to avoid such a massive bloat.

in hindsight that's what I should I have done. I got annoyed by red CI and just fixed the issues here.

@jstuczyn jstuczyn force-pushed the bugfix/surb-refresh branch from 759957e to 1054567 Compare July 4, 2025 14:54
@jstuczyn jstuczyn merged commit 8331143 into release/2025.12-dolcelatte Jul 4, 2025
19 of 22 checks passed
@jstuczyn jstuczyn deleted the bugfix/surb-refresh branch July 4, 2025 15:29
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