Opened 19 years ago

Closed 19 years ago

Last modified 19 years ago

#2312 closed defect (fixed)

E-mail validator should accept new TLDs such as ".museum"

Reported by: anonymous Owned by: Adrian Holovaty
Component: contrib.admin Version: dev
Severity: minor 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

Id doesn't accept new TLDs like .museum

Change History (5)

comment:1 by Adrian Holovaty, 19 years ago

Resolution: → wontfix
Status: new → closed
Summary: Admin e-mail validation problem. → E-mail validator should accept new TLDs such as ".museum"

comment:2 by anonymous, 19 years ago

why is this a wontfix?

comment:3 by anonymous, 19 years ago

interesting, i would like to know as well.

changing

r')@(?:[A-Z0-9-]+\.)+[A-Z]{2,4}$', re.IGNORECASE)  # domain

to

r')@(?:[A-Z0-9-]+\.)+[A-Z]{2,6}$', re.IGNORECASE)  # domain

you get

email_re = re.compile(
    r"(^[-!#$%&'*+/=?^_`{}|~0-9A-Z]+(\.[-!#$%&'*+/=?^_`{}|~0-9A-Z]+)*"  # dot-atom
    r'|^"([\001-\010\013\014\016-\037!#-\[\]-\177]|\\[\001-011\013\014\016-\177])*"' # quoted-string
    r')@(?:[A-Z0-9-]+\.)+[A-Z]{2,6}$', re.IGNORECASE)  # domain

which should work

comment:4 by Chris Beaven, 19 years ago

Resolution: wontfix
Status: closed → reopened

I see no reason why this shouldn't be fixed, .museum is a valid TLD.

Adrian, would you care to explain why?

comment:5 by Adrian Holovaty, 19 years ago

Resolution: → fixed
Status: reopened → closed

(In [3305]) Fixed #2312 -- E-mail validator now accepts TLDs up to 6 characters long

Note: See TracTickets for help on using tickets.
Back to Top