Download & Extend

Link module treats .asia as an invalid domain name

Project:Link
Version:5.x-2.5
Component:Code
Category:bug report
Priority:normal
Assigned:jcfiala
Status:closed (fixed)

Issue Summary

... but in fact, it should be valid.

I checked the list at http://www.iana.org/domains/root/db/ to see if there were others that might be missing, and the only one I found was .tel - however, that seems to be a particularly weird one according to http://en.wikipedia.org/wiki/.tel and might not be worth including? So for now, the attached patch just adds .asia to the list.

AttachmentSize
link_asia_valid_domain.patch777 bytes

Comments

#1

Assigned to:Anonymous» jcfiala
Status:needs review» patch (to be ported)

Yeah, that looks good to me. I'll try to get that into cvs soon. Good catch.

#2

Status:patch (to be ported)» fixed

Okay, this is in the 6 branch...

#3

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

#4

Version:6.x-2.5» 5.x-2.5
Status:closed (fixed)» reviewed & tested by the community

Looks like the patch applies just as well to the Drupal 5 branch...

#5

Status:reviewed & tested by the community» fixed

Now fixed in the 5.x branch.

#6

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nobody click here