[Mailman-Developers] List-* header suggestion

Barry A. Warsaw bwarsaw@beopen.com
Thu, 5 Oct 2000 15:44:57 -0400 (EDT)


>>>>> "CVR" == Chuq Von Rospach <chuqui@plaidworks.com> writes:

    CVR> they aren't on my site, ron -- it' spossible you have an MTA
    CVR> that *is* folding them for you. Does it do laundry, too?

    CVR> Folding them would be a good idea, IMHO.

Me too.  Here's the patch.

-Barry

-------------------- snip snip --------------------
Index: CookHeaders.py
===================================================================
RCS file: /cvsroot/mailman/mailman/Mailman/Handlers/CookHeaders.py,v
retrieving revision 1.15
diff -u -r1.15 CookHeaders.py
--- CookHeaders.py	2000/09/25 13:40:19	1.15
+++ CookHeaders.py	2000/10/05 19:42:51
@@ -17,6 +17,7 @@
 """Cook a message's Subject header.
 """
 
+import string
 import re
 import urlparse
 from Mailman import mm_cfg
@@ -115,6 +116,11 @@
     # one copy of each, and we want to be sure it's ours.
     for h, v in headers.items():
         del msg[h]
+        # Wrap these lines if they are too long.  78 character width probably
+        # shouldn't be hardcoded.  The adding of 2 is for the colon-space
+        # separator.
+        if len(h) + 2 + len(v) > 78:
+            v = string.join(string.split(v, ', '), ',\n\t')
         msg[h] = v
     #
     # Always delete List-Archive header, but only add it back if the list is