server icon indicating copy to clipboard operation
server copied to clipboard

Enforce valid email address for User email

Open harminius opened this issue 1 year ago • 1 comments

resolves #2537

Add additional format validity check for the user's email.

  • when the validation registration form
  • on the database level

Registration form check Screenshot from 2024-10-25 09-25-56

Before running the migration, we need to sanitize our database.

To see which emails break the new rule, try select email from public."user" where email !~ '^[A-Za-z0-9._%+-]+@[A-Za-z0-9.-]+.[A-Za-z]{2,}$';

harminius avatar Oct 25 '24 13:10 harminius

I am not confident with this change and multi language support.

MarcelGeo avatar Oct 28 '24 15:10 MarcelGeo

@harminius I suggest also to check db against invalid characters and non-utf8 mess, if you can pass me the queries

varmar05 avatar Dec 10 '24 07:12 varmar05

Pull Request Test Coverage Report for Build 13589212467

Details

  • 22 of 22 (100.0%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.03%) to 92.436%

Totals Coverage Status
Change from base Build 13584997311: 0.03%
Covered Lines: 6795
Relevant Lines: 7351

💛 - Coveralls

github-actions[bot] avatar Feb 28 '25 14:02 github-actions[bot]