388729 - Support CAPAB IDENTIFY-MSG as used on freenode

Status: RESOLVED DUPLICATE of bug 687798

(Other Applications Graveyard :: ChatZilla, enhancement)

Description

[John L. Villalovos]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.4) Gecko/20070509 SeaMonkey/1.1.2
Build Identifier: 

http://freenode.net/news-2003-05-19.shtml

irc.freenode.net have a feature/capability called 'CAPAB IDENTIFY-MSG'.  The
identify-msg capability causes the first character of a msg or notice to be set
to a + if the user is identified with nickserve and a - if not.

This can work right now by doing a: /quote capab identify-msg
On the server and you can see what happens.  A channel with lots and lots of
activity to see this in action is irc://irc.freenode.net/ubuntu

Unfortunately this seems to break use of: /me action

So my request for enhancement is the ability of Chatzilla to take advantage of
this capability and color the display differently depending on if a user is
identified or not identified.

Thanks,
John


Reproducible: Didn't try

Comment 1

[John L. Villalovos]

Here is a script for irssi which adds support for this feature.  Not sure if it would help someone implement it or not.

http://web.archive.org/web/20040604065734/http://svn.ben.reser.org/format_identify/releases/current/format_identify.pl

Comment 2

[Glen Mailer]

Attachment: handle identify-msg without breaking

I had to implement this in irc.js as handler.js was too far up the event propagation tree to do this remotely neatly.

This patch merely fixes the broken ctcp ACTION messages by stripping the +/- off the message and setting the property "identifiedMessage" on the event.

This property is not currently displayed in any way (so I suppose there's not much point in turning it on actually...)

I cant remember how we decided this should be displayed after discussing it on IRC, if anyone has any suggestions, feel free to mention them.

Comment 3

[James Ross]

Comment on attachment 333641 [details] [diff] [review]
handle identify-msg without breaking

You use capab.identifyMsg and e.identifiedMessage; I'd prefer these to have the same name. Otherwise, it looks fine.

Comment 4

[Glen Mailer]

Attachment: [checked in] consistent variable names

Comment 5

[David Vo [:auscompgeek]]

freenode now uses CAP REQ instead of CAPAB, as of the upgrade to ircd-seven. Should this bug be morphed into "support CAP REQ IDENTIFY-MSG", or should that be a new bug?

Comment 6

[David Vo [:auscompgeek]]

identify-msg has been deprecated for the new account-notify capability. Should this bug be WONTFIXed?