#822 list-id might violate RFC2919 and RFC2822

[Bernhard Reiter]

Code in
http://cvs.sourceforge.net/viewcvs.py/mailman/mailman/Mailman/Handlers/CookHeaders.py?rev=2.33.2.5&view=markup
(as shipped with mailman 2.1.4)
will use formataddr for the List-Id:

listid_h = formataddr((desc, listid))

This will violate the RFC2919, if not(desc),
because the angle-brackets might be left out.

3. The List-Id Header Field

The List-Id header MAY optionally include a
description by including
it as a "phrase" [DRUMS] before the
angle-bracketed list identifier.

The syntax of the List-Id header follows:
list-id-header = "List-ID:" [phrase] "<"
list-id ">" CRLF

Unlike most headers in [RFC822],
the List-Id header does not allow free insertion of
whitespace and comments around tokens.

[Go inside ~mailman/bin]
python

Python 2.1.3 (#1, Sep 7 2002, 15:29:56)
[GCC 2.95.4 20011002 (Debian prerelease)] on linux2
Type "copyright", "credits" or "license" for more
information.
>>> import paths
>>> from email.Utils import formataddr
>>> formataddr(("A strange mailing
list","strange.list.org"))
'A strange mailing list <strange.list.org>'
>>> formataddr((None,"strange.list.org"))
'strange.list.org'
>>> formataddr(("","strange.list.org"))
'strange.list.org'

Solution: go back to old code and
check
http://sourceforge.net/tracker/download.php?group_id=103&atid=100103&file_id=57904&aid=665732
to limit the line length so you don't violate RFC2822

[Bernhard Reiter]

Logged In: YES
user_id=113859

I'm raising the priority to 6 because
many people get annoyed if the List-ID headers change.
Mailman 2.1.3 did better in the RFC2919 regard,
2.1.4 breaks it in an attempt to fix other bugs.

[Bernhard Reiter]

Logged In: YES
user_id=113859

Added a fix for the problem:

[ 933762 ] fix for #880073 list-id can violate RFC2919
https://sourceforge.net/tracker/index.php?func=detail&aid=933762&group_id=103&atid=300103

[Tokio Kikuchi]

Logged In: YES
user_id=67709

This was already fixed in CVS/2.1.6b