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

CommentFileSizeAuthor
link_asia_valid_domain.patch777 bytesDavid_Rothstein

Comments

jcfiala’s picture

Assigned: Unassigned » 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.

jcfiala’s picture

Status: Patch (to be ported) » Fixed

Okay, this is in the 6 branch...

Status: Fixed » Closed (fixed)

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

David_Rothstein’s picture

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

jcfiala’s picture

Status: Reviewed & tested by the community » Fixed

Now fixed in the 5.x branch.

Status: Fixed » Closed (fixed)

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