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

Closed
opened 2026-03-03 02:06:23 +03:00 by kerem · 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
kerem 2026-03-03 02:06:23 +03:00
  • closed this issue
  • added the
    bug
    label
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.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/vaultwarden#1116
No description provided.