-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
[Bug] E-Mail Login is dependent on capitalization #1740
Comments
Hey! @Eikeike While signing up, it stores the email ID as it is provided by the user. |
See Wiki
While it is a common practice to keep the naming convention case-insensitive, the RFC itself says it is case-sensitive and there are a few mail servers around which allow case-sensitive naming. So, IMHO we should keep this |
Good discussion. As much as the spec of email addresses in general say they are case-sensitive, I'd prefer to enhance user experience. I'll take a peek into this issue, or if anyone's willing to take up this issue, that would be great! A possible solution would be simply to .toLowerCase() the input provided by the user for username and email, before inserting it into the DB. And also, it would need to be normalised again when checking the credentials (in Local Auth Strategy). |
@AmruthPillai can i work on this ?? i will do those two things , you have mentioned above ?? |
hello @AmruthPillai , can I go with this issue |
hello @AmruthPillai, is this issue still open?? |
@AmruthPillai I was wondering about the statement you put forward earlier, could you please guide me further taking the quote below as the "beginning" for further conversation. Regards
|
Is there an existing issue for this?
Product Variant
Cloud (http://rxresu.me)
Current Behavior
When you register with your e-mail address and there is a capital letter in the mail address, there will be a server error when you login with the same e-mail with a different capitalization
Expected Behavior
E-Mail login and registration should be completely indepentend from capitalization in the e-mail address.
Steps To Reproduce
What browsers are you seeing the problem on?
Chrome
What template are you using?
None
Anything else?
No response
The text was updated successfully, but these errors were encountered: