Skip to content

fix(bruteforce): Don't throttle requests with failing CSRF - #61686

Merged
AndyScherzinger merged 1 commit into
masterfrom
bug-dont-throttle-csrf
Jul 7, 2026
Merged

fix(bruteforce): Don't throttle requests with failing CSRF#61686
AndyScherzinger merged 1 commit into
masterfrom
bug-dont-throttle-csrf

Conversation

@kesselb

@kesselbkesselb commented Jul 1, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

Don't throttle requests with failing CSRF

Follow up for #49560 to re-introduce #46392

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@kesselb
kesselb requested a review from a team as a code ownerJuly 1, 2026 09:09
@kesselb
kesselb requested review from Altahrim, come-nc, icewind1991 and leftybournes and removed request for a teamJuly 1, 2026 09:09
@kesselbkesselb self-assigned this Jul 1, 2026
@kesselbkesselb added bug 3. to review Waiting for reviews labels Jul 1, 2026
@kesselbkesselb added this to the Nextcloud 35 milestone Jul 1, 2026
@kesselb

kesselb commented Jul 1, 2026

Copy link
Copy Markdown
ContributorAuthor

/backport to stable34

@kesselb

kesselb commented Jul 1, 2026

Copy link
Copy Markdown
ContributorAuthor

/backport to stable33

@kesselb
kesselbforce-pushed the bug-dont-throttle-csrf branch from 5d2ec68 to 4e1978bCompareJuly 3, 2026 08:36
@kesselb
fix(bruteforce): Don't throttle requests with failing CSRF
58b21c0
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb
kesselbforce-pushed the bug-dont-throttle-csrf branch from 4e1978b to 58b21c0CompareJuly 7, 2026 15:02
@kesselb
kesselb enabled auto-merge July 7, 2026 15:22
@AndyScherzinger
AndyScherzinger merged commit 65736e5 into masterJul 7, 2026
195 of 197 checks passed
@AndyScherzinger
AndyScherzinger deleted the bug-dont-throttle-csrf branch July 7, 2026 16:34
This was referenced Jul 7, 2026
Merged
Merged
@nextcloud-botnextcloud-bot mentioned this pull request Aug 11, 2026
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AltahrimAltahrimAltahrim approved these changes
@benjaminfruehbenjaminfruehbenjaminfrueh approved these changes
@icewind1991icewind1991Awaiting requested review from icewind1991icewind1991 is a code owner automatically assigned from nextcloud/server-backend
@leftybournesleftybournesAwaiting requested review from leftybournesleftybournes is a code owner automatically assigned from nextcloud/server-backend
@come-nccome-ncAwaiting requested review from come-nccome-nc is a code owner automatically assigned from nextcloud/server-backend

Assignees

@kesselbkesselb

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Milestone

Nextcloud 35

Development

Successfully merging this pull request may close these issues.

4 participants

@kesselb@Altahrim@benjaminfrueh@AndyScherzinger