Skip to content

feat(core): allow switching 2FA method from the challenge page - #64754

Open
skjnldsv wants to merge 1 commit into
masterfrom
feature/2fa-challenge-ux
Open

skjnldsv wants to merge 1 commit into
masterfrom
feature/2fa-challenge-ux

Conversation

@skjnldsv

@skjnldsv skjnldsv commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Summary

Once a 2FA method is picked on the selection page, the challenge page only offers "Use backup code" and "Cancel login". If I pick Nextcloud Notification but have no logged-in device at hand, the only way to use TOTP instead is "Cancel login", which logs out and means typing the password again.

This adds a "Use another method" link to the challenge page that goes back to /login/selectchallenge, keeping redirect_url. The pending 2FA session is untouched since selectChallenge is a plain GET, so nothing changes on the session side.

The link only shows when there is another regular provider than the current one. With only TOTP + backup codes, the existing "Use backup code" link already covers it, so it stays hidden. On the backup codes page it shows as soon as a regular provider exists.

Screenshot

image

(Ill do a design improvement of the whole 2FA in a secondary PR)

Checklist

AI (if applicable)

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

Show a "Use another method" link on the challenge page when another
regular provider is available, so users can go back to the selection
page instead of cancelling the login.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
@skjnldsv
skjnldsv marked this pull request as ready for review September 25, 2026 07:57
@skjnldsv
skjnldsv requested review from Altahrim, come-nc, icewind1991 and sorbaugh and removed request for a team September 25, 2026 07:57
@skjnldsv skjnldsv added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 25, 2026
@skjnldsv
skjnldsv added this pull request to stack #64756 September 25, 2026 08:34

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant