linkifying of 3rd level domains #24

Closed
vitalyvolkov opened this Issue May 21, 2011 · 3 comments

Comments

Projects
None yet
2 participants

Bleach has ignored only 2nd domain emails like www@example.com, but if domain is 3rd level it will be linkified.

In [5]: bleach.linkify('www@example.com.ua')
Out[5]: u'www@example.<a href="http://com.ua" rel="nofollow">com.ua</a>'

Expected: ignoring 3rd level domains.

Member

jsocol commented Jun 8, 2011

I honestly don't know how to fix this. Maybe the TLD list needs to include things like co.uk instead of just uk when the TLD is never used by itself?

SmileyChris added a commit to SmileyChris/bleach that referenced this issue Jun 9, 2011

Add fix for linkifying of 3rd level domains (jsocol/#24) and add link…
…ifying

of "complete" urls (i.e. those containing user:pass@ sections).
Member

jsocol commented Jun 10, 2011

@jsocol jsocol closed this Jun 10, 2011

Nice feature! Thanks for that!

jsocol pushed a commit that referenced this issue Jun 10, 2011

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