search

gregjacobs / Autolinker.js

Utility to Automatically Link URLs, Email Addresses, Phone Numbers, Twitter handles, and Hashtags in a given block of text/HTML
MIT License
1.48k stars 238 forks source link

Bug in autolink #218

Closed shubham-agrawal closed 5 years ago

shubham-agrawal commented 6 years ago

http://search%3Fp%5BsubCategory%5D%3Dpen+drive%26f%5BMemory%5D%3D8%2520GB

should either highlight the complete url as link or nothing at all as it is not a valid url but is only highlights http://search

gregjacobs commented 5 years ago

Not sure how that's a valid URL at all (it's not, really), but seems GitHub's naïve autolinking implementation thinks it's okay. However, notice that clicking on that link in your browser brings you nowhere.

I'm going to say this is not something we want to accept as a URL in Autolinker.