Skip to content

fix(core): Throttle lost-password reset form when link is disabled#61615

Merged
AndyScherzinger merged 1 commit into
masterfrom
fix/lostpassword-throttle-reset-form
Jul 7, 2026
Merged

fix(core): Throttle lost-password reset form when link is disabled#61615
AndyScherzinger merged 1 commit into
masterfrom
fix/lostpassword-throttle-reset-form

Conversation

@nfebe

@nfebe nfebe commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

When password reset links were disabled, the reset form did not rate-limit failed token attempts, leaving that endpoint open to unthrottled token guessing. Failed attempts on the form are now always throttled, consistent with the other reset steps.

@nfebe nfebe requested a review from sorbaugh June 26, 2026 19:01
@nfebe nfebe requested a review from a team as a code owner June 26, 2026 19:01
@nfebe nfebe requested review from ArtificialOwl and removed request for a team June 26, 2026 19:01
@nfebe nfebe added the 3. to review Waiting for reviews label Jun 26, 2026
@kesselb

kesselb commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

/backport to stable34

@kesselb

kesselb commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

/backport to stable33

When password reset links were disabled, the reset form did not rate-limit
failed token attempts, leaving that endpoint open to unthrottled token
guessing. Failed attempts on the form are now always throttled, consistent
with the other reset steps.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
@kesselb kesselb force-pushed the fix/lostpassword-throttle-reset-form branch from ec38edf to f9e274d Compare July 7, 2026 14:53
@AndyScherzinger AndyScherzinger merged commit a256d7e into master Jul 7, 2026
243 of 250 checks passed
@AndyScherzinger AndyScherzinger deleted the fix/lostpassword-throttle-reset-form branch July 7, 2026 16:56
@AndyScherzinger AndyScherzinger added this to the Nextcloud 35 milestone Jul 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants