361914 - hostmasks should not be formatted as mailto urls

Status: VERIFIED FIXED

(Other Applications :: ChatZilla, defect)

Description

[Stephanie Daugherty [:sdaugherty]]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0

Host masks on IRC rarely if ever correspond to email addresses. The formatting preference to hyper link text that looks like an email address should only apply to  text that's likely to contain an email address, and not to server responses that will contain only host masks.

This functionality makes it harder to copy and paste host masks in the default client configuration.

Reproducible: Always

Comment 1

[:Gijs (he/him)]

Marking as blocking for some bugs that need updated patches when this gets in.

Comment 2

[:Gijs (he/him)]

Attachment: Patch

Disables mailto munging by default, enabling for echo, msg, notice, ctcp-action, topic (because the topic might contain mailto's).

Comment 3

[James Ross]

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

>     munger.addRule ("mailto",
>        /(?:\s|\W|^)((mailto:)?[^<>\[\]()\'\"\s\u201d]+@[^.<>\[\]()\'\"\s\u201d]+\.[^<>\[\]()\'\"\s\u201d]+)/i,
>-                    insertMailToLink);
>+                    insertMailToLink, false);

This wont work as you intend; the code at the end of initMunger enables it from the pref straight away. You probably want to rename the rule to ".mailto".

r=silver with that.

Comment 4

[:Gijs (he/him)]

Attachment: Patch for checkin

Patch for checkin. Note that I've also did the whole munger entry disable stuff in output-window.js, so the channel header always gets updated correctly

Comment 5

[:Gijs (he/him)]

Checked in --> FIXED

Comment 6

[Stephanie Daugherty [:sdaugherty]]

Latest nightly seems to work correctly.

Comment 7

[:Gijs (he/him)]

(In reply to comment #6)
> Latest nightly seems to work correctly.

--> VERIFIED

Comment 8

[:Gijs (he/him)]

This change has regressed multiline sends, because this line:

client.currentObject.dispatch("say " + msg);

fails for multiline messages because the commands get split onto lines which don't have a command, which fails to work.

The proper way to do this is:

client.currentObject.dispatch("say", {message: msg});

which fixes the regression. That fix has been checked in with r+ from Silver on IRC.