JackieChiles/Cinch

Turn URLs in chat into links

Closed this issue · 2 comments

When URLs are entered into the chat window, they should show up as clickable links.

Note that this will only work with URLs prefixed with http:// or https://. Testing prevention of script injections here would be appreciated. If you can successfully cause an injection let me know.

Looks good. I'm not seeing a way to cause an injection. The .text attribute
protects against that very well, and adding JS to the end of a proper URL
does nothing.

On Thu, Oct 16, 2014 at 6:48 PM, JackieChiles notifications@github.com
wrote:

Note that this will only work with URLs prefixed with http:// or https://.
Testing prevention of script injections here would be appreciated. If you
can successfully cause an injection let me know.


Reply to this email directly or view it on GitHub
#158 (comment).