Link module treats .asia as an invalid domain name

David_Rothstein - April 16, 2009 - 21:32
Project:Link
Version:5.x-2.5
Component:Code
Category:bug report
Priority:normal
Assigned:jcfiala
Status:reviewed & tested by the community
Description

... 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

#1

jcfiala - April 16, 2009 - 21:59
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

jcfiala - April 20, 2009 - 03:27
Status:patch (to be ported)» fixed

Okay, this is in the 6 branch...

#3

System Message - May 4, 2009 - 03:31
Status:fixed» closed

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

#4

David_Rothstein - May 4, 2009 - 03:40
Version:6.x-2.5» 5.x-2.5
Status:closed» reviewed & tested by the community

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

 
 

Drupal is a registered trademark of Dries Buytaert.