[GH-ISSUE #1963] Inviting users with capital letters in there mail address without mail enabled will fail to register #13904

Closed
opened 2026-04-23 05:37:33 -05:00 by GiteaMirror · 2 comments
Owner

Originally created by @frankenstein91 on GitHub (Sep 6, 2021).
Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/1963

Inviting users with capital letters in there mail address without mail enabled will fail to register

see Matrix chat for more information

Originally created by @frankenstein91 on GitHub (Sep 6, 2021). Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/1963 Inviting users with capital letters in there mail address without mail enabled will fail to register see Matrix chat for more information
GiteaMirror added the bug label 2026-04-23 05:37:33 -05:00
Author
Owner

@BlackDex commented on GitHub (Sep 6, 2021):

Confirmed.
The issue is that the mail addresses are not stored with lower-case letters within the invitations table.
Here: https://github.com/dani-garcia/vaultwarden/blob/main/src/db/models/user.rs#L352-L356 there is no conversion to lower-case, while here, https://github.com/dani-garcia/vaultwarden/blob/main/src/db/models/user.rs#L76-L79 it does.

And here are two different data paths used: https://github.com/dani-garcia/vaultwarden/blob/main/src/api/admin.rs#L269-L274

We need to make sure that all that info is lower-case stored and checked, and that the same data source is used for all items.

<!-- gh-comment-id:913677567 --> @BlackDex commented on GitHub (Sep 6, 2021): Confirmed. The issue is that the mail addresses are not stored with lower-case letters within the invitations table. Here: https://github.com/dani-garcia/vaultwarden/blob/main/src/db/models/user.rs#L352-L356 there is no conversion to lower-case, while here, https://github.com/dani-garcia/vaultwarden/blob/main/src/db/models/user.rs#L76-L79 it does. And here are two different data paths used: https://github.com/dani-garcia/vaultwarden/blob/main/src/api/admin.rs#L269-L274 We need to make sure that all that info is lower-case stored and checked, and that the same data source is used for all items.
Author
Owner

@BlackDex commented on GitHub (Sep 6, 2021):

Workaround for now: Invite users only with lower-case e-mail addresses.

<!-- gh-comment-id:913678397 --> @BlackDex commented on GitHub (Sep 6, 2021): Workaround for now: Invite users only with lower-case e-mail addresses.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vaultwarden#13904