-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 for Logging In is Case Sensitive #7
Comments
I totally agree with @LIV3N. |
I think you will have to replace var result = await _signInManager.PasswordSignInAsync(model.Email.ToLower(),
model.MasterPasswordHash,
model.Device?.ToDevice()); with var result = await _signInManager.PasswordSignInAsync(model.Email,
model.MasterPasswordHash,
model.Device?.ToDevice()); I am not confident enough to make a PR. Maybe with Unit Tests. ;) |
Logging in is not suppose to be case sensitive. I have put things in place to handle this already (at least I thought). What client application are you seeing this in? |
I got the error on web, Chrome, and Android. If you need any more info, let me know. On Mon, Oct 10, 2016, 19:17 Kyle Spearrin notifications@github.com wrote:
|
@LIV3N from what client did you do the initial registration from? |
I found the bug. The issue occurred when registering from the web vault. If you currently have an account that is locked up because of this let me know and I can purge it out for you to re-register. Thanks for the report! fix is here: bitwarden/web@d49d227 |
Ephemeral Environment Details: |
Created account using email with caps, for example test@GMAIL.com.
Unable to login with test@GMAIL.com, had to log in as test@gmail.com.
Email authentication should not be case sensitive.
The text was updated successfully, but these errors were encountered: