bpepple / metron

Django website for a comic book database
https://metron.cloud/
GNU General Public License v3.0
62 stars 7 forks source link

Email address with + sign is rejected as invalid #249

Closed PeteS4 closed 3 months ago

PeteS4 commented 3 months ago

Describe the bug I tried registering with my perfectly valid email address that contains a plus sign (something like abc+def@example.org). The address is rejected as invalid.

To Reproduce Steps to reproduce the behavior:

  1. Go to https://metron.cloud/accounts/signup/
  2. Enter data including an email address with a + sign
  3. Click 'Sign up'
bpepple commented 3 months ago

Glanced at the logs with our e-mail verification service, and the only failure I see gave this response:

{
  "valid": false,
  "block": true,
  "disposable": false,
  "domain": "posteo.net",
  "text": "Invalid e-mail",
  "reason": "E-mail format is invalid",
  "risk": 10,
  "mx_host": null,
  "possible_typo": null,
  "mx_ip": null,
  "mx_info": null,
  "last_changed_at": "2024-05-26T16:08:39+00:00"
}

I know other accounts have a + in the email address in the past, but maybe our service no longer allows them. You might try removing the + filter hack and see if the works.

PeteS4 commented 3 months ago

Yes, I removed the filter part and it worked fine .. I guess it's just a pet peeve of mine that an increasing number of web pages have this unnecessary hurdle 😌