Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Email case sensitivity #378

Open
smartspot2 opened this issue Jan 19, 2023 · 0 comments · May be fixed by #385
Open

Email case sensitivity #378

smartspot2 opened this issue Jan 19, 2023 · 0 comments · May be fixed by #385
Labels
bug Something isn't working sev3 High severity - Production system impaired

Comments

@smartspot2
Copy link
Member

When creating new users, the email field seems to be case sensitive, allowing for two users to be created where one has the email "user@berkeley.edu" and the other with "User@berkeley.edu". This runs into a major issue with the Google OAuth, as the authentication system detects two different users with the same email, throwing an error, not allowing the user to log in and view anything at all.

This should be explored a little more though, as it seems like by formal specification, it's up to the mail server whether they view emails as case-sensitive or not (everything after the @ is always mandated to be lower case). This means Django's EmailField would understandably be case-sensitive as well.

@smartspot2 smartspot2 added the bug Something isn't working label Jan 19, 2023
@smartspot2 smartspot2 added the sev3 High severity - Production system impaired label Jan 26, 2023
@edwardneo edwardneo self-assigned this Feb 3, 2023
@edwardneo edwardneo linked a pull request Feb 3, 2023 that will close this issue
@yzdu02 yzdu02 self-assigned this Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working sev3 High severity - Production system impaired
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants