search

kylewm / redwind

Flask-based personal website #IndieWeb #OwnYourData
Other
45 stars 5 forks source link

Increase TLD max length from 4 to 14 in link regex #68

Closed LanaCoyote closed 9 years ago

LanaCoyote commented 9 years ago

This allows us to autolink new ICANN domain names.

14 because the longest domain I can find is .cancerresearch

kylewm commented 9 years ago

honestly this just makes the problem worse (in my experience), since there are so many more false positives (there are already a lot just with 4 characters). I would prefer to shift this code over to use brevity to autolink since it has a comprehensive list of all the actual TLDs and checks against them.

LanaCoyote commented 9 years ago

Alright, I've made this into an issue at #69 instead.