mirror of
https://github.com/dani-garcia/vaultwarden.git
synced 2026-04-26 01:35:54 +03:00
[GH-ISSUE #1963] Inviting users with capital letters in there mail address without mail enabled will fail to register #1116
Labels
No labels
SSO
Third party
better for forum
bug
bug
documentation
duplicate
enhancement
future Vault
future Vault
future Vault
good first issue
help wanted
low priority
notes
pull-request
question
troubleshooting
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/vaultwarden#1116
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
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
@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.
@BlackDex commented on GitHub (Sep 6, 2021):
Workaround for now: Invite users only with lower-case e-mail addresses.