Change the handling of login errors. #2010

New Issue

MrUnknownDE · 2026-04-06T02:49:20+02:00

MrUnknownDE commented

2026-04-06 02:49:20 +02:00

Originally created by @BlackDex on 9/6/2022

Previously FlashMessage was used to provide an error message during login.
This PR changes that flow to not use redirect for this, but renders the HTML and responds using the correct status code where needed. This should solve some issues which were reported in the past.

Thanks to @RealOrangeOne, for initiating this with a PR.

Fixes #2448
Fixes #2712
Closes #2715

*Originally created by @BlackDex on 9/6/2022* Previously FlashMessage was used to provide an error message during login. This PR changes that flow to not use redirect for this, but renders the HTML and responds using the correct status code where needed. This should solve some issues which were reported in the past. Thanks to @RealOrangeOne, for initiating this with a PR. Fixes #2448 Fixes #2712 Closes #2715

MrUnknownDE closed this issue

2026-04-06 02:49:20 +02:00

Sign in to join this conversation.

No Label

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/vaultwarden#2010