Not recognizing newer top-level domains #163

Closed
Berzerker opened this Issue Aug 23, 2014 · 3 comments

Projects

None yet

3 participants

@Berzerker

Example here: http://imgur.com/DJpf199

Seems like the newer top level domains are confusing putty a bit.

@FauxFaux FauxFaux added a commit that referenced this issue Nov 9, 2014
@FauxFaux GH-163: update "sensible" regex with new tlds
This is probably a losing battle.  Maybe I should just give in and recommend people use the "nearly any" option.

$ echo $(grep '^\.' tlds | cut -c2- | egrep -v '^[a-z]{2}$' | sort | uniq) | fold -w 61 -s | tr ' ' '|' | sed 's/^/"/;s/$/"/'
f61c710
@FauxFaux
Owner
FauxFaux commented Nov 9, 2014

I've added loads more things to the list.

This is probably a losing battle. Maybe I should just give in and recommend people use the "nearly any" option.

@FauxFaux FauxFaux closed this Nov 9, 2014
@Berzerker

Ok, well, with the new update, a lot of non-standard TLDs, and even ones that were working before (like bit.ly) have stopped being recognized, changing to "nearly any" works.

@incognico

@Berzerker you could use some more generic regex like (([a-zA-Z]+://|[wW][wW][wW]\.)[^ '")>]+)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment