GNU bug report logs - #28831
27.0.50; debbugs: funny address in To field

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 28831 in the body.
You can then email your comments to 28831 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: Tino Calancha <tino.calancha <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 27.0.50; debbugs: funny address in To field
Date: Sat, 14 Oct 2017 18:45:47 +0900

X-Debbugs-CC: michael.albinus <at> gmx.de
X-Debbugs-CC: larsi <at> gnus.org

Recently when using debbugs to list the bug reports,
I see a new address in the 'To' as 'xxxxx <at> https:'
with xxxxx the bug number.  For instance, this is the
headers from Bug#24000:

From: "Greg A. Woods" <woods <at> robohack.ca>
Subject: bug#24000: 24.5; makefile-bsdmake-mode has incomplete keyword syntax
To: 24000 <at> debbugs.gnu.org, 24000 <at> https:
Date: Fri, 15 Jul 2016 15:28:43 -0700 (1 year, 12 weeks, 6 days ago)
Resent-From: "Greg A. Woods" <woods <at> robohack.ca>
Reply-To: "Greg A. Woods" <woods <at> robohack.ca>

When I reply such reports the funny address appear in CC.
What are those '24000 <at> https:'? Do we need them?

In GNU Emacs 27.0.50 (build 16, x86_64-pc-linux-gnu, GTK+ Version 3.22.11)
 of 2017-10-14
Repository revision: dcb86584fd12fc44d246bbe033372693785c127d

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

From: Noam Postavsky <npostavs <at> users.sourceforge.net>
To: Tino Calancha <tino.calancha <at> gmail.com>
Cc: 28831 <at> debbugs.gnu.org, larsi <at> gnus.org
Subject: Re: bug#28831: 27.0.50; debbugs: funny address in To field
Date: Thu, 19 Oct 2017 20:29:55 -0400

[Message part 1 (text/plain, inline)]

tags 28831 + patch
quit

Tino Calancha <tino.calancha <at> gmail.com> writes:

> When I reply such reports the funny address appear in CC.
> What are those '24000 <at> https:'? Do we need them?

Hah, I thought I was the only one (somehow this report did not land in
my inbox, I only happened to find it when looking at the bug list).  The
following seems to fix it, but there is some caching.  When I open this
bug the "https:" is still there, but opening other bugs that I haven't
opened in the current session yet then it's not.  I'm not sure where the
cache is kept.

[0001-Handle-https-url-for-debbugs-mbox-Bug-28831.patch (text/x-diff, inline)]

From e1c49aec252ac8e79631c96fe68d2dcc725bdde4 Mon Sep 17 00:00:00 2001
From: Noam Postavsky <npostavs <at> gmail.com>
Date: Thu, 19 Oct 2017 20:07:05 -0400
Subject: [PATCH] Handle https url for debbugs mbox (Bug#28831)

In 2017-09-13 "Prefer HTTPS to FTP and HTTP in documentation",
gnus-bug-group-download-format-alist was updated to use https for the
debbugs.gnu.org mbox links, but gnus-read-ephemeral-bug-group assumed
http links.
* lisp/gnus/gnus-group.el (gnus-read-ephemeral-bug-group): Strip
https, not http.
---
 lisp/gnus/gnus-group.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/gnus/gnus-group.el b/lisp/gnus/gnus-group.el
index 985efe6272..259d046e9e 100644
--- a/lisp/gnus/gnus-group.el
+++ b/lisp/gnus/gnus-group.el
@@ -2469,7 +2469,7 @@ gnus-read-ephemeral-bug-group
 	       (format "%s@%s" (car ids)
 		       (replace-regexp-in-string
 			"/.*$" ""
-			(replace-regexp-in-string "^http://" "" mbox-url)))))
+			(replace-regexp-in-string "^https://" "" mbox-url)))))
 	  (goto-char (point-min))
 	  (while (re-search-forward (concat "^" message-unix-mail-delimiter)
 				    nil t)
-- 
2.11.0

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

From: Tino Calancha <tino.calancha <at> gmail.com>
To: Noam Postavsky <npostavs <at> users.sourceforge.net>
Cc: 28831 <at> debbugs.gnu.org, larsi <at> gnus.org
Subject: Re: bug#28831: 27.0.50; debbugs: funny address in To field
Date: Fri, 20 Oct 2017 11:14:33 +0900

Noam Postavsky <npostavs <at> users.sourceforge.net> writes:

> tags 28831 + patch
> quit
>
> Tino Calancha <tino.calancha <at> gmail.com> writes:
>
>> When I reply such reports the funny address appear in CC.
>> What are those '24000 <at> https:'? Do we need them?
>

> The following seems to fix it, but there is some caching.  When I open this
> bug the "https:" is still there, but opening other bugs that I haven't
> opened in the current session yet then it's not.  I'm not sure where the
> cache is kept.
Excellent!  It works here.
Thank you.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Noam Postavsky <npostavs <at> users.sourceforge.net>
Cc: 28831 <at> debbugs.gnu.org, Tino Calancha <tino.calancha <at> gmail.com>
Subject: Re: bug#28831: 27.0.50; debbugs: funny address in To field
Date: Fri, 20 Oct 2017 11:35:14 +0200

Noam Postavsky <npostavs <at> users.sourceforge.net> writes:

> -			(replace-regexp-in-string "^http://" "" mbox-url)))))
> +			(replace-regexp-in-string "^https://" "" mbox-url)))))

Perhaps that should be

> +			(replace-regexp-in-string "^https?://" "" mbox-url)))))


-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

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

From: Noam Postavsky <npostavs <at> users.sourceforge.net>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 28831 <at> debbugs.gnu.org, Tino Calancha <tino.calancha <at> gmail.com>
Subject: Re: bug#28831: 27.0.50; debbugs: funny address in To field
Date: Fri, 20 Oct 2017 08:55:52 -0400

[Message part 1 (text/plain, inline)]

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> Noam Postavsky <npostavs <at> users.sourceforge.net> writes:
>
>> -			(replace-regexp-in-string "^http://" "" mbox-url)))))
>> +			(replace-regexp-in-string "^https://" "" mbox-url)))))
>
> Perhaps that should be
>
>> +			(replace-regexp-in-string "^https?://" "" mbox-url)))))

Or maybe we could use url-parse?  Seems a bit neater.

[v2-0001-Handle-https-url-for-debbugs-mbox-Bug-28831.patch (text/x-diff, inline)]

From 992b5b8a33eab5ea0afa020030408f76dcfd4330 Mon Sep 17 00:00:00 2001
From: Noam Postavsky <npostavs <at> gmail.com>
Date: Thu, 19 Oct 2017 20:07:05 -0400
Subject: [PATCH v2] Handle https url for debbugs mbox (Bug#28831)

In 2017-09-13 "Prefer HTTPS to FTP and HTTP in documentation",
gnus-bug-group-download-format-alist was updated to use https for the
debbugs.gnu.org mbox links, but gnus-read-ephemeral-bug-group assumed
http links.
* lisp/gnus/gnus-group.el (gnus-read-ephemeral-bug-group): Use
url-parse functions to get the host name, instead of ad-hoc regexps.
---
 lisp/gnus/gnus-group.el | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/lisp/gnus/gnus-group.el b/lisp/gnus/gnus-group.el
index 985efe6272..4a41c49590 100644
--- a/lisp/gnus/gnus-group.el
+++ b/lisp/gnus/gnus-group.el
@@ -2467,9 +2467,7 @@ gnus-read-ephemeral-bug-group
 	;; Add the debbugs address so that we can respond to reports easily.
 	(let ((address
 	       (format "%s@%s" (car ids)
-		       (replace-regexp-in-string
-			"/.*$" ""
-			(replace-regexp-in-string "^http://" "" mbox-url)))))
+                       (url-host (url-generic-parse-url mbox-url)))))
 	  (goto-char (point-min))
 	  (while (re-search-forward (concat "^" message-unix-mail-delimiter)
 				    nil t)
-- 
2.11.0

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Noam Postavsky <npostavs <at> users.sourceforge.net>
Cc: 28831 <at> debbugs.gnu.org, Tino Calancha <tino.calancha <at> gmail.com>
Subject: Re: bug#28831: 27.0.50; debbugs: funny address in To field
Date: Fri, 20 Oct 2017 15:53:28 +0200

Noam Postavsky <npostavs <at> users.sourceforge.net> writes:

> Or maybe we could use url-parse?  Seems a bit neater.

Yeah, sounds good.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

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

From: Noam Postavsky <npostavs <at> users.sourceforge.net>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 28831 <at> debbugs.gnu.org, Tino Calancha <tino.calancha <at> gmail.com>
Subject: Re: bug#28831: 27.0.50; debbugs: funny address in To field
Date: Sat, 21 Oct 2017 15:52:36 -0400

tags 28831 fixed
close 28831 26.1
quit

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> Noam Postavsky <npostavs <at> users.sourceforge.net> writes:
>
>> Or maybe we could use url-parse?  Seems a bit neater.
>
> Yeah, sounds good.

Pushed to emacs-26.

[1: b060e091c3]: 2017-10-21 15:19:14 -0400
  Handle https url for debbugs mbox (Bug#28831)
  https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=b060e091c35e8482e0e091a214cf5afe5039bd0e>

Previous Next

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