Opened 9 years ago
Closed 9 years ago
#24349 closed Cleanup/optimization (fixed)
Limit URL domain name labels to 63 characters
| Reported by: | Anoop Thomas Mathew | Owned by: | nobody |
|---|---|---|---|
| Component: | Core (Other) | Version: | dev |
| Severity: | Normal | Keywords: | gTLD, email validation |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
Newly added gTLDs can be longer upto 63 characters. Need to accommodate for that as well while validating the email.
Ref: http://en.wikipedia.org/wiki/Domain_Name_System#cite_ref-rfc1034_1-2
Change History (3)
comment:1 by , 9 years ago
| Patch needs improvement: | set |
|---|---|
| Summary: | Support domain names with gTLDs like .consulting, .company, .website etc → Limit URL domain name labels to 63 characters |
| Triage Stage: | Unreviewed → Accepted |
| Type: | Bug → Cleanup/optimization |
comment:2 by , 9 years ago
| Patch needs improvement: | unset |
|---|---|
| Triage Stage: | Accepted → Ready for checkin |
Note:
See TracTickets
for help on using tickets.
Unless I made a mistake in reviewing this... as noted on the pull request, these domains seem to validate just fine. The proposed regex does limit TLDs to 63 characters per the RFC which could be useful.