#26204 closed Bug (fixed)
URLValidator does not accept TLDs with "-"s
| Reported by: | Luke Faraone | Owned by: | Tim Graham |
|---|---|---|---|
| Component: | Core (Other) | Version: | 1.8 |
| Severity: | Normal | Keywords: | regression |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description (last modified by )
Forwarded from:
http://bugs.debian.org/814372
Previous versions of Django allowed for "-"s in the rightmost component of a hostname. This was tightened in a previous release. While the case for punycode domains was fixed, there's no technical reason to disallow them in any part of a hostname component except for at the beginning or end.
That is to say: http://ubuntu-mirror.my-tld/ubuntu is flagged as an invalid URL, even though the DNS has an IP for it, and apt happily upgrades from the archive mirror that is there.
The fix for <https://code.djangoproject.com/ticket/25452#comment:2> appears to be when this regression was introduced.
Change History (7)
comment:1 by , 9 years ago
| Description: | modified (diff) |
|---|
comment:2 by , 9 years ago
| Description: | modified (diff) |
|---|
comment:3 by , 9 years ago
| Component: | Uncategorized → Core (Other) |
|---|---|
| Owner: | changed from to |
| Status: | new → assigned |
| Triage Stage: | Unreviewed → Accepted |
comment:4 by , 9 years ago
| Has patch: | set |
|---|
Note:
See TracTickets
for help on using tickets.
PR