#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 , 18 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 , 18 years ago
comment:3 by , 18 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 , 18 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 , 18 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Note:
See TracTickets
for help on using tickets.
why is this a wontfix?