From e337e914137e89ce8c58a8e66ae21d0591727d3b Mon Sep 17 00:00:00 2001
From: Mark Sapiro
Date: Wed, 11 Apr 2012 15:24:10 -0700
Subject: Fixed the RFC 1153 digest footer to be compliant.

---
 src/mailman/runners/digest.py | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

(limited to 'src/mailman/runners/digest.py')

diff --git a/src/mailman/runners/digest.py b/src/mailman/runners/digest.py
index afc11f732..c347667f8 100644
--- a/src/mailman/runners/digest.py
+++ b/src/mailman/runners/digest.py
@@ -284,16 +284,17 @@ class RFC1153Digester(Digester):
                         self._mlist.fqdn_listname, 
                         self._mlist.digest_footer_uri))
                 footer_text = ''                
-            # This is not strictly conformant RFC 1153.  The trailer is only
-            # supposed to contain two lines, i.e. the "End of ... Digest" line
-            # and the row of asterisks.  If this screws up MUAs, the solution
-            # is to add the footer as the last message in the RFC 1153 digest.
-            # I just hate the way that VM does that and I think it's confusing
-            # to users, so don't do it unless there's a clamor.
+            # MAS: There is no real place for the digest_footer in an RFC 1153
+            # compliant digest, so add it as an additional message with
+            # Subject: Digest Footer
             print >> self._text, self._separator30
             print >> self._text
+            print >> self._text, 'Subject: ' + _('Digest Footer')
+            print >> self._text
             print >> self._text, footer_text
             print >> self._text
+            print >> self._text, self._separator30
+            print >> self._text
         # Add the sign-off.
         sign_off = _('End of ') + self._digest_id
         print >> self._text, sign_off
-- 
cgit v1.2.3-70-g09d2