Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#31959 closed Cleanup/optimization (wontfix)

EmailValidator must not accept hyphen as first character

Reported by: James Pic Owned by: nobody
Component: Utilities Version: 3.1
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Getting the following exception:

Aug 28 10:57:22 production 83a5b18c66bc[1752]: smtplib.SMTPRecipientsRefused: {'-redacted@…': (501, b'5.1.3 Bad recipient address syntax')}

EmailValidator should not accept a hyphen as first caracter.

Note: Component set to "Utilities" but really I believe a "Validator" Component should be available.

Change History (2)

comment:1 by Mariusz Felisiak, 4 years ago

Resolution: wontfix
Status: newclosed
Summary: EmailValidator must not accept hyphen as first caracterEmailValidator must not accept hyphen as first character
Type: UncategorizedCleanup/optimization

Thanks for this ticket, however we discussed a similar case in #25452 and the consensus on the mailing list was to simplify the validation, not make it more comprehensive.

comment:2 by James Pic, 4 years ago

Actually, I couldn't confirm that emails must not start with an hyphen, so this issue is invalid.

I think the solution is to have an email validator that will actually open a connection to the smtp server and call smtp vrfy command.

Last edited 4 years ago by James Pic (previous) (diff)
Note: See TracTickets for help on using tickets.
Back to Top