Processed: Re: Bug#975690: lintian: detect invalid Uploaders fields that are missing separating commas

2020-11-25 Thread Debian Bug Tracking System
Processing control commands:

> block -1 by 509935
Bug #975690 [lintian] lintian: detect invalid Uploaders fields that are missing 
separating commas
975690 was not blocked by any bugs.
975690 was not blocking any bugs.
Added blocking bug(s) of 975690: 401452 and 509935

-- 
975690: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=975690
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#975690: lintian: detect invalid Uploaders fields that are missing separating commas

2020-11-25 Thread Felix Lechner
Control: block -1 by 509935

Hi.

On Tue, Nov 24, 2020 at 10:51 PM Paul Wise  wrote:
>
> Please detect this and emit an error about it, probably it should also
> get onto the ftp-master lintian reject list.

Lintian uses the standard Perl module Email::Address::XS [1] to parse
all address fields. (There are others, but this was most capable in
parsing Unicode "phrases".) The module states that it "parses input
string[s] sequentially according to RFC 5322 grammar." In the given
package, the last two uploaders are simply discarded:

$ bin/lintian -T mail-contact -T trimmed-field --no-tag-display-limit
/mirror/debian/pool/main/y/yubico-piv-tool/yubico-piv-tool_2.1.1-2.dsc
C: yubico-piv-tool source: mail-contact Maintainer "Debian
Authentication Maintainers"

C: yubico-piv-tool source: mail-contact Uploaders "Alessio Di Mauro"

C: yubico-piv-tool source: mail-contact Uploaders "Dain Nilsson"

C: yubico-piv-tool source: trimmed-field Maintainer Debian
Authentication Maintainers

C: yubico-piv-tool source: trimmed-field Uploaders Alessio Di Mauro
, Dain Nilsson  Klas Lindfors
, nicoo ,
[some output omitted]

Perhaps the RFC 5322 grammar is too lenient with respect to errors,
and focuses instead on sending to at least some recipients.

Kind regards,
Felix Lechner

[1] https://metacpan.org/pod/Email::Address::XS#parse