Skip to content

fix(2fa): allow null redirect URL on 2FA challenge page#57908

Merged
nickvergessen merged 1 commit intomasterfrom
fix/2fa/allow-null-redirect-url-challenge
Jan 29, 2026
Merged

fix(2fa): allow null redirect URL on 2FA challenge page#57908
nickvergessen merged 1 commit intomasterfrom
fix/2fa/allow-null-redirect-url-challenge

Conversation

@ChristophWurst
Copy link
Copy Markdown
Member

  • Resolves: #

Summary

Makes it possible to use 2FA during login without a redirect URL.

Before After
Bildschirmfoto vom 2026-01-29 14-36-53 Bildschirmfoto vom 2026-01-29 14-37-00

Regression of #55474.
Discovered in nextcloud/twofactor_totp#1716

TODO

  • Fix

Checklist

Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
@ChristophWurst ChristophWurst added this to the Nextcloud 34 milestone Jan 29, 2026
@ChristophWurst ChristophWurst self-assigned this Jan 29, 2026
@ChristophWurst ChristophWurst requested a review from a team as a code owner January 29, 2026 13:41
@ChristophWurst ChristophWurst requested review from leftybournes and removed request for a team January 29, 2026 13:41
@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 29, 2026
@nickvergessen nickvergessen merged commit 2066eb4 into master Jan 29, 2026
171 of 173 checks passed
@nickvergessen nickvergessen deleted the fix/2fa/allow-null-redirect-url-challenge branch January 29, 2026 14:30
@nextcloud-bot nextcloud-bot mentioned this pull request Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants