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

In my company many (many) url starts like https://edomus.tesa// . . . . but the link module says that it is an invalid URL.

I would be good, if it was configurable.

Thanks.

Comments

dqd’s picture

Status: Active » Closed (won't fix)
freebug’s picture

Status: Closed (won't fix) » Needs review

Hi I am facing the same issue.

Our client wants the URL with domains ".sbu" to be acceptable.

There needs to be a settings page where additional domains can be added.

Otherwise, can the domain validation be turned off on per field basis.

For the time being I have added the "sbu" domain in the "LINK_DOMAINS" constant.

Thanks

jcfiala’s picture

Status: Needs review » Active

Hm. I thought it was configurable, but apparently at some point I was mistaken.

I'll try to put this on my list of things to do, but I'd be more than happy for someone to come up with a patch that makes it configurable. I think this is something that isn't so much per-field as per module, myself.

In the short term, adding a domain to the LINK_DOMAINS is a perfectly decent hack.

Setting to 'active' because there's nothing to review that I can see.

bcurry’s picture

Issue summary: View changes

As per jcfiala's recommendation I was able to make the .bike domain valid for use within my site by adding "bike" to the following line in the link.module file.

define('LINK_DOMAINS', 'aero|arpa|asia|biz|com|cat|coop|edu|gov|info|int|jobs|mil|museum|name|nato|net|org|pro|travel|mobi|local|xxx|bike');

Anonymous’s picture

Status: Active » Closed (duplicate)

All of the separate issues concerning validation failure with newer top-level domains are resolved by the patch at https://www.drupal.org/node/1846202. Rather than adding TLDs one by one to the LINK_DOMAINS constant, that patch actually updates from an official list of active TLDs from IANA, so as new TLDs are approved, they will automatically become valid in this module. Please go test & review the patch & mark as RTBC.