Removed LOWER() from email check

[mikoczy]

No description provided.

[markoph]

@mikoczy please add description to your PR. Why do you need to remove this validation check?

This transformation to lower case was added so we don't register two users with same email address - eg. john@gmail.com and JOHN@gmail.com. Gmail (and most of the email providers) is not case sensitive so these emails are actually same account.

If you have valid use case we can move this validation to some kind of validation provider which could be disabled. But we don't want to remove it.

[mikoczy]

@markoph sorry, it is a follow up to an older pr (#6).. mysql is case insensitive too, so the use case you described, cannot happen... on the other hand, when using LOWER, mysql will not use the email index, so if the users table is big enough, this query can be very slow, timeouts can happen, when adding a user through api..

[markoph]

@mikoczy FYI: your commit was merged internally and changes will be part of next release.

[mikoczy]

@markoph great, thanks