Duo: use the formatted db email #1033

New Issue

MrUnknownDE · 2026-04-06T01:34:10+02:00

MrUnknownDE commented

2026-04-06 01:34:10 +02:00

Originally created by @Timshel on 7/25/2024

Was rebasing and had a conflict with the latest addition of validate_duo_login.
Had made the fix for the OpenID PR and never back-ported it (Don't exactly remember the issue and it's buried in the comments, but it was linked to using the non lowercased email).

I think in general it makes more sense if available to use the reference email from the user instead of the non formatted value from the form.
I don't have a Duo account anymore so can't really test it :(.

*Originally created by @Timshel on 7/25/2024* Was rebasing and had a conflict with the latest addition of `validate_duo_login`. Had made the fix for the OpenID PR and never back-ported it (Don't exactly remember the issue and it's buried in the comments, but it was linked to using the non lowercased email). I think in general it makes more sense if available to use the reference email from the user instead of the non formatted value from the form. I don't have a Duo account anymore so can't really test it :(.

MrUnknownDE closed this issue

2026-04-06 01:34:10 +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#1033