372874 - Clicking on URL with a @ in it opens MailNews Compose (or default mail handler?)

Status: RESOLVED FIXED

(Other Applications :: ChatZilla, defect)

Description

[Frank Wein [:mcsmurf]]

To reproduce:
1. Fetch a CZ trunk build
2. Send the URL http://www.google.com/groups?selm=[e-mail] as message to a channel
3. Click on that URL

Results:
In SeaMonkey a MailNews Compose window opens and "http://www.google.com/groups" is inserted as To: address.

Comment 1

[:Gijs (out for now; he/him)]

Attachment: Patch

I like simple patches.
I like really simple patches even better.

What I like best though, are patches where you add just one single character, and that magically "makes it work".

Comment 2

[James Ross]

Comment on attachment 257869 [details] [diff] [review]
Patch

Saved by RFC822. :)

Technically, though, the following are also invalid in that part: ",", ";", "\", ".". Clearly the last one can't be safely banned, thanks to the laxity of past programs. The others might be worth considering.

Comment 3

[:Gijs (out for now; he/him)]

I've added ; and \ to the mix. I didn't want to try to check if , was in use, but guessed that if . was, some idiot(s) would probably have used , as well. Oh well.

Checking in mozilla/extensions/irc/xul/content/mungers.js;
/cvsroot/mozilla/extensions/irc/xul/content/mungers.js,v  <--  mungers.js
new revision: 1.4; previous revision: 1.3
done