Autolinker.js Linking text before protocol

Linking text before protocol

Open mbuzzard opened this issue 8 years ago • 4 comments

Found a bug where the following string was including the text before the protocol in the link:

DE:https://foo.bar.com/foo/bar

Mar 04 '16 19:03 mbuzzard

Hey, interesting. Thought that was handled. Will check it out.

Mar 08 '16 05:03 gregjacobs

The above example seems to be fixed now, but here's another case where it's linking text before the protocol:

Hello...https://foo.bar.com/foo/bar

https://tonicdev.com/56e9b1511f9bd21200fa4786/56e9b1511f9bd21200fa4787

Mar 16 '16 19:03 hongrich

Nov 23 '16 11:11 jasongrishkoff

In 1.4.2,

Hello...https://foo.bar.com/foo/bar

does not link anything. I think it should link the second part.

Mar 03 '17 02:03 olafleur