-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
Bug or feature? #5626
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
Labels
state:duplicate
Duplicate of already reported issue
Comments
Not sure if it is a bug but to avoid this problem you could just have |
Yes, but if I do that I will need to also perform a .lower when the user is created and their email is saved, right?
On May 30, 2019, at 3:06 PM, Tom Fox <[email protected]<mailto:[email protected]>> wrote:
Not sure if it is a bug but to avoid this problem you could just have userEmail.lower
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#5626?email_source=notifications&email_token=AJ5GWY5KTEBKDGJGHCHBB7LPYAQR7A5CNFSM4HRH3TVKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWTHESI#issuecomment-497447497>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AJ5GWYZOJ5VIXL4NV7WSBJ3PYAQR7ANCNFSM4HRH3TVA>.
|
That’s right, you could do that in a beforeSave trigger on the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Issue Description
From our iOS App we are using this call to log in a user:
PFUser.logInWithUsername(inBackground: userEmail, password: userPassword)
But if the user enters their userEmail with a different case it fails to log in. So is this a bug or is there a way to turn of case sensitive compare for the login? It would be good if I could pass a case flag of something but I do not see a part for that.
Steps to reproduce
Create a user, for example: [email protected] then try to log in and enter [email protected] and it fails to log in.
Expected Results
Actual Outcome
Environment Setup
Server
Database
Logs/Trace
The text was updated successfully, but these errors were encountered: