[BUG - Develop] False positives on URL detection

Describe the bug

If someone e.g. says “Awww.” in-game, the pattern www.X gets picked up as a web address by the text2html parser.

To Reproduce

Steps to reproduce the behavior:

1. Open up the webclient for an active game.
2. emote www.t
3. See error

Expected behavior

The URL detection for the text2html parser needs to be reworked to avoid false positives, or possibly removed entirely. I’m of the opinion to remove it entirely. If I take on fixing this, which is likely, that’s the approach I’ll take.

Develop-branch commit

3992f37da

[aMiss-aWry]

One reason to do it server side could be parsing URLs into clickable links via MXP for clients that support it.

[InspectorCaracal]

I don’t have a strong opinion here, but why URLs specifically? You could just send pure ANSI over the websocket and have all of the ANSI to HTML conversion live client-side.

There’s nothing about URLs specifically, but recognizing an unmarked string of text and deciding to apply formatting to it is very different from converting existing and intentional formatting tags to a different style of formatting tags. Currently, URLs are the only thing that have that kind of process applied - and it’s broken, so the cleanest solution to me is “don’t do it”.

One reason to do it server side could be parsing URLs into clickable links via MXP for clients that support it.

Yeah, if it was done server-side I think this would be the way to do it.