Skip to content

🐛 fix(auth): bound password derivations - #1863

Merged
gaborbernat merged 1 commit into
mainfrom
fix/password-queue-1466
Aug 30, 2026
Merged

gaborbernat merged 1 commit into
mainfrom
fix/password-queue-1466

Conversation

@gaborbernat

Copy link
Copy Markdown
Member

Argon2 work now uses one process-wide admission budget for running and queued derivations. Requests beyond the fixed four-active/four-waiting bound fail before identity lookup or hashing, while a canceled queued request releases its slot and a canceled running request remains charged until blocking work exits.

Every password-derived operation shares the same path, including decoys and stale-verifier upgrades. Public HTTP and distributed callers return service unavailable without exposing whether an account exists.

Closes #1466

Reject password work before hashing when the process-wide active and waiting
capacity is full. Propagate overloads as service unavailable without
distinguishing real users from decoys.

Refs #1466
@gaborbernat gaborbernat added the bug Something isn't working label Aug 30, 2026
@gaborbernat gaborbernat self-assigned this Aug 30, 2026
@codspeed

codspeed Bot commented Aug 30, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 23 untouched benchmarks
⏩ 139 skipped benchmarks1


Comparing fix/password-queue-1466 (8f06f6c) with main (60698d6)

Open in CodSpeed

Footnotes

  1. 139 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 peryx | 🛠️ Build #34294861 | 📁 Comparing 8f06f6c against latest (60698d6)

  🔍 Preview build  

1 file changed
± core/access/authentication/index.html

@gaborbernat
gaborbernat merged commit b8436e6 into main Aug 30, 2026
32 checks passed
@gaborbernat
gaborbernat deleted the fix/password-queue-1466 branch August 30, 2026 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bound queued password derivations

1 participant