ejabberd - Comments for "mod_muc: recognise more URI schemes in logged HTML (htmlize())?"

The patch was commited to

mfoss — Tue, 17 Jul 2007 08:28:34 +0000

The patch was commited to ejabberd SVN. So next version of ejabberd will have this enhancement.

I updated your patch to

mfoss — Tue, 03 Jul 2007 17:44:04 +0000

I updated your patch to ejabberd SVN and submitted here: Recognise more URI schemes in logged HTML.

I guess it will be applied soon.

afterthought

qu1j0t3 — Sun, 01 Jul 2007 00:08:06 +0000

'>' should probably end a URL also.

something like this?

qu1j0t3 — Sat, 30 Jun 2007 20:26:48 +0000

I think this works okay. I included single quote as well as your suggestions.

--- ejabberd-1.1.3-dist/src/mod_muc/mod_muc_log.erl     2006-09-22 03:58:58.000000000 -0400
+++ ejabberd-1.1.3/src/mod_muc/mod_muc_log.erl  2007-06-30 16:21:17.066776505 -0400
@@ -657,7 +657,7 @@
     S2 = element(2, regexp:gsub(S1, "\\&", "\\&")),
     S3 = element(2, regexp:gsub(S2, "<", "\\<")),
     S4 = element(2, regexp:gsub(S3, ">", "\\>")),
-    S5 = element(2, regexp:gsub(S4, "(http|ftp)://.[^ ]*", "<a href=\"&\">&</a>")),
+    S5 = element(2, regexp:gsub(S4, "[-+.a-zA-Z0-9]+://[^] )\'\"}]+", "<a href=\"&\">&</a>")),
     S5.
 
 get_room_info(RoomJID, Opts) ->

There are a lot of schemes

mfoss — Tue, 05 Jun 2007 14:29:53 +0000

There are a lot of schemes at IANA, and maybe more on the future. So I think it's better to simply check for lexically valid scheme. Do you have knowledge/interest enought to try to write such a patch?

I found another related problem, maybe you can/want try to fix this too: when a message has an URI inside brackets, like:

Check the JSF site (http://www.jabber.org)

the generated HTML does an ugly thing with the ending bracket:

Check the JSF site (<a href="http://www.jabber.org)">http://www.jabber.org)</a>

I guess it should handle correctly characters like "", (), [], {}...

If you will not, tell me and I'll try on the following days.