GNU bug report logs - #39345
28.0.50; rcirc-authenticate-before-join due to case sensitive mismatch

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 39345 in the body.
You can then email your comments to 39345 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jake Nelson <jake.nelson <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 28.0.50; rcirc-authenticate-before-join due to case sensitive mismatch
Date: Wed, 29 Jan 2020 09:29:26 -0500

1. join irc.freenode.net with a lowercase nick "examplenick"
2. authenticate to nickserv
3. nickserv echos back  "-NickServ- You are now identified for
"Examplenick" capitalizing the nickname.
4. rcirc-authenticate-before-join does not trigger, so no channels are
auto-joined.

The existing check is case sensitive.  The patch below fixes the issue
for me on freenode.

diff --git a/lisp/net/rcirc.el b/lisp/net/rcirc.el
index 5722582ab6..b516c8d0a6 100644
--- a/lisp/net/rcirc.el
+++ b/lisp/net/rcirc.el
@@ -2624,12 +2624,12 @@ rcirc-check-auth-status
         (when (or
                (and ;; nickserv
                 (string= sender "NickServ")
-                (string= target rcirc-nick)
-                (member message
+                (string= (downcase target) (downcase rcirc-nick))
+                (member (downcase message)
                         (list
-                         (format "You are now identified for
\C-b%s\C-b." rcirc-nick)
-                        (format "You are successfully identified as
\C-b%s\C-b." rcirc-nick)
-                         "Password accepted - you are now recognized."
+                         (downcase (format "You are now identified
for \C-b%s\C-b." rcirc-nick))
+                        (downcase (format "You are successfully
identified as \C-b%s\C-b." rcirc-nick))
+                         (downcase "Password accepted - you are now
recognized.")
                          )))
                (and ;; quakenet
                 (string= sender "Q")

Message #8 received at 39345 <at> debbugs.gnu.org (full text, mbox):

From: Noam Postavsky <npostavs <at> gmail.com>
To: Jake Nelson <jake.nelson <at> gmail.com>
Cc: 39345 <at> debbugs.gnu.org
Subject: Re: bug#39345: 28.0.50;
 rcirc-authenticate-before-join due to case sensitive mismatch
Date: Wed, 29 Jan 2020 10:20:46 -0500

Jake Nelson <jake.nelson <at> gmail.com> writes: 

> 1. join irc.freenode.net with a lowercase nick "examplenick" 
> 2. authenticate to nickserv 3. nickserv echos back  "-NickServ- 
> You are now identified for "Examplenick" capitalizing the 
> nickname. 

Hmm, I use "npostavs" as my nick on freenode, and I get 

   09:43 -NickServ- You are now identified for npostavs.

Maybe the difference is that you are registering with the capitalized
version, but then authenticating with lowercase?

For reference, it looks like ERC also uses case-sensitive comparison
(see erc-nickserv-identification-autodetect and erc-nickserv-alist).

Message #11 received at 39345 <at> debbugs.gnu.org (full text, mbox):

From: Jake Nelson <jake.nelson <at> gmail.com>
To: Noam Postavsky <npostavs <at> gmail.com>
Cc: 39345 <at> debbugs.gnu.org
Subject: Re: bug#39345: 28.0.50; rcirc-authenticate-before-join due to case
 sensitive mismatch
Date: Wed, 29 Jan 2020 12:59:21 -0500

You are correct. To test, I dropped my registration with NickServ, and
re-registered with my nickname lowercase, and now see it lowercased in
the "You are now identified for [nickname]" message.

Re-registering resolves this issue for me, but it is not apparent
rcirc-authenticate-before-join is case sensitive when NickServ itself
does not care.




On Wed, Jan 29, 2020 at 10:20 AM Noam Postavsky <npostavs <at> gmail.com> wrote:
>
> Jake Nelson <jake.nelson <at> gmail.com> writes:
>
> > 1. join irc.freenode.net with a lowercase nick "examplenick"
> > 2. authenticate to nickserv 3. nickserv echos back  "-NickServ-
> > You are now identified for "Examplenick" capitalizing the
> > nickname.
>
> Hmm, I use "npostavs" as my nick on freenode, and I get
>
>     09:43 -NickServ- You are now identified for npostavs.
>
> Maybe the difference is that you are registering with the capitalized
> version, but then authenticating with lowercase?
>
> For reference, it looks like ERC also uses case-sensitive comparison
> (see erc-nickserv-identification-autodetect and erc-nickserv-alist).

Message #14 received at 39345 <at> debbugs.gnu.org (full text, mbox):

From: Noam Postavsky <npostavs <at> gmail.com>
To: Jake Nelson <jake.nelson <at> gmail.com>
Cc: 39345 <at> debbugs.gnu.org
Subject: Re: bug#39345: 28.0.50;
 rcirc-authenticate-before-join due to case sensitive mismatch
Date: Thu, 12 Mar 2020 21:30:40 -0400

tags 39345 fixed
close 39345 28.1
quit

Jake Nelson <jake.nelson <at> gmail.com> writes:

> You are correct. To test, I dropped my registration with NickServ, and
> re-registered with my nickname lowercase, and now see it lowercased in
> the "You are now identified for [nickname]" message.
>
> Re-registering resolves this issue for me, but it is not apparent
> rcirc-authenticate-before-join is case sensitive when NickServ itself
> does not care.

Yes, since NickServ is case-insensitive, it makes sense that rcirc
should be as well.  I pushed a slightly different patch to master to do
this.

[1: 3758ff0f3a]: 2020-03-12 21:06:28 -0400
  rcirc: Match NickServ messages case-insensitively (Bug#39345)
  https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=3758ff0f3ad365b7a56c3e63a8d0d5f00f3d5085

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.