* bug#39345: 28.0.50; rcirc-authenticate-before-join due to case sensitive mismatch
@ 2020-01-29 14:29 Jake Nelson
2020-01-29 15:20 ` Noam Postavsky
0 siblings, 1 reply; 4+ messages in thread
From: Jake Nelson @ 2020-01-29 14:29 UTC (permalink / raw)
To: 39345
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")
^ permalink raw reply related [flat|nested] 4+ messages in thread
* bug#39345: 28.0.50; rcirc-authenticate-before-join due to case sensitive mismatch
2020-01-29 14:29 bug#39345: 28.0.50; rcirc-authenticate-before-join due to case sensitive mismatch Jake Nelson
@ 2020-01-29 15:20 ` Noam Postavsky
2020-01-29 17:59 ` Jake Nelson
0 siblings, 1 reply; 4+ messages in thread
From: Noam Postavsky @ 2020-01-29 15:20 UTC (permalink / raw)
To: Jake Nelson; +Cc: 39345
Jake Nelson <jake.nelson@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).
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#39345: 28.0.50; rcirc-authenticate-before-join due to case sensitive mismatch
2020-01-29 15:20 ` Noam Postavsky
@ 2020-01-29 17:59 ` Jake Nelson
2020-03-13 1:30 ` Noam Postavsky
0 siblings, 1 reply; 4+ messages in thread
From: Jake Nelson @ 2020-01-29 17:59 UTC (permalink / raw)
To: Noam Postavsky; +Cc: 39345
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@gmail.com> wrote:
>
> Jake Nelson <jake.nelson@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).
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#39345: 28.0.50; rcirc-authenticate-before-join due to case sensitive mismatch
2020-01-29 17:59 ` Jake Nelson
@ 2020-03-13 1:30 ` Noam Postavsky
0 siblings, 0 replies; 4+ messages in thread
From: Noam Postavsky @ 2020-03-13 1:30 UTC (permalink / raw)
To: Jake Nelson; +Cc: 39345
tags 39345 fixed
close 39345 28.1
quit
Jake Nelson <jake.nelson@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
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-03-13 1:30 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-01-29 14:29 bug#39345: 28.0.50; rcirc-authenticate-before-join due to case sensitive mismatch Jake Nelson
2020-01-29 15:20 ` Noam Postavsky
2020-01-29 17:59 ` Jake Nelson
2020-03-13 1:30 ` Noam Postavsky
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.