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

Emails should be normalised, as the Postgres database is case sensative #35

Closed
JonPSmith opened this issue May 21, 2022 · 0 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@JonPSmith
Copy link
Owner

JonPSmith commented May 21, 2022

In the current version of AuthP (3.2.0) the email of a user is stored as the user provides it. But an email is NOT case sensitive. This will be a problem as Postgres is case sensative.

The simplest way to do this is add a .Lower to the email in the AuthUser class

Things to do

  • Add a .Lower to the email in the AuthUser class (note: do after the username has as been set). DONE
  • Apply .Lower to the email in the AuthUsersAdmin FindAuthUserByEmailAsync method. DONE
  • Change SyncUsers to normalise Emails. DONE
  • Fix all tests containing emails. DONE
  • Change each Example to say that "NOTE: The email username is also the password".
@JonPSmith JonPSmith added the bug Something isn't working label May 21, 2022
@JonPSmith JonPSmith self-assigned this May 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant