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

Inconsistent email case-sensitivity #345

Open
slint opened this issue Sep 23, 2020 · 0 comments
Open

Inconsistent email case-sensitivity #345

slint opened this issue Sep 23, 2020 · 0 comments

Comments

@slint
Copy link
Member

slint commented Sep 23, 2020

Describe the bug

The case-sensitivity of the email field is inconsistent throughout the sign-up, login, and settings pages.

Steps to Reproduce

  1. Go to an Invenio instance without being logged-in
  2. Sign up and put any part of the email address in upper-case letters (e.g. [email protected])
  3. After that go to the user profile account settings page and try to change your username or full name
  4. An error is returned saying [email protected] is already associated with an account.

This is probably because the email address is looked-up or compared in a case-sensitive fashion.

Expected behavior

The common consensus is that email addresses SHOULD be treated as case-insensitive. There are two approaches on this:

  • Convert and store all email address input to lowercase
  • Store the original user input email address but use case-insensitive methods for lookups (e.g. a lower-case column index)

Screenshots (if applicable)

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants