GNU bug report logs - #34293
27.0.50; comma inserted at the beginning of email address

Previous Next

Package: gnus;

Reported by: Juri Linkov <juri <at> linkov.net>

Date: Sat, 2 Feb 2019 21:18:01 UTC

Severity: normal

Tags: fixed

Found in version 5.13

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

Bug is archived. No further changes may be made.

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

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

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bugs <at> gnus.org:
bug#34293; Package gnus. (Sat, 02 Feb 2019 21:18:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Juri Linkov <juri <at> linkov.net>:
New bug report received and forwarded. Copy sent to bugs <at> gnus.org. (Sat, 02 Feb 2019 21:18:01 GMT) Full text and rfc822 format available.

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

From: Juri Linkov <juri <at> linkov.net>
To: bug-gnu-emacs <at> gnu.org
Subject: 27.0.50; comma inserted at the beginning of email address
Date: Sat, 02 Feb 2019 22:50:24 +0200
X-Debbugs-Package: gnus
X-Debbugs-Version: 5.13

I have customized 'message-alternative-emails' to a regexp that
matches email aliases, but when a email lacks the 'To:' field,
message-use-alternative-email-as-from doesn't remove
the leading ',' prepended due to the empty 'To' field:

(defun message-use-alternative-email-as-from ()
  ...
  (let* ((fields '("To" "Cc" "From"))
	 (emails
	  (message-tokenize-header
	   (mail-strip-quoted-names
	    (mapconcat 'message-fetch-reply-field fields ","))))
  ...

As a result, a comma is added to the beginning of email address.

Commit 357ae5dba5f from 2016-02-08 is a relatively recent change,
and it seems that this bug crept in from this commit.




Information forwarded to bugs <at> gnus.org:
bug#34293; Package gnus. (Tue, 09 Jul 2019 17:21:02 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Juri Linkov <juri <at> linkov.net>
Cc: 34293 <at> debbugs.gnu.org
Subject: Re: bug#34293: 27.0.50;
 comma inserted at the beginning of email address
Date: Tue, 09 Jul 2019 19:20:06 +0200
Juri Linkov <juri <at> linkov.net> writes:

> I have customized 'message-alternative-emails' to a regexp that
> matches email aliases, but when a email lacks the 'To:' field,
> message-use-alternative-email-as-from doesn't remove
> the leading ',' prepended due to the empty 'To' field:
>
> (defun message-use-alternative-email-as-from ()
>   ...
>   (let* ((fields '("To" "Cc" "From"))
> 	 (emails
> 	  (message-tokenize-header
> 	   (mail-strip-quoted-names
> 	    (mapconcat 'message-fetch-reply-field fields ","))))
>   ...
>
> As a result, a comma is added to the beginning of email address.

Yup.  Should be fixed on the trunk now.

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




Added tag(s) fixed. Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Tue, 09 Jul 2019 17:21:02 GMT) Full text and rfc822 format available.

bug closed, send any further explanations to 34293 <at> debbugs.gnu.org and Juri Linkov <juri <at> linkov.net> Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Tue, 09 Jul 2019 17:21:03 GMT) Full text and rfc822 format available.

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Wed, 07 Aug 2019 11:24:11 GMT) Full text and rfc822 format available.

This bug report was last modified 4 years and 234 days ago.

Previous Next


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