[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

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



Control: block -1 by 509935

Hi.

On Tue, Nov 24, 2020 at 10:51 PM Paul Wise <pabs@debian.org> 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"
<pkg-auth-maintainers@lists.alioth.debian.org>
C: yubico-piv-tool source: mail-contact Uploaders "Alessio Di Mauro"
<alessio@yubico.com>
C: yubico-piv-tool source: mail-contact Uploaders "Dain Nilsson"
<dain@yubico.com>
C: yubico-piv-tool source: trimmed-field Maintainer Debian
Authentication Maintainers
<pkg-auth-maintainers@lists.alioth.debian.org>
C: yubico-piv-tool source: trimmed-field Uploaders Alessio Di Mauro
<alessio@yubico.com>, Dain Nilsson <dain@yubico.com> Klas Lindfors
<klas@yubico.com>, nicoo <nicoo@debian.org>,
[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


Reply to: