------------------------------------------------------------
revno: 1473
fixes bug: https://launchpad.net/bugs/1313146
committer: Mark Sapiro <m...@msapiro.net>
branch nick: 2.1
timestamp: Sat 2014-04-26 21:25:34 -0700
message:
  Most Mailman generated notices to list owners and moderators are now
  sent as Precedence: list instead of bulk.  (LP: #1313146)
modified:
  Mailman/Message.py
  NEWS


--
lp:mailman/2.1
https://code.launchpad.net/~mailman-coders/mailman/2.1

Your team Mailman Checkins is subscribed to branch lp:mailman/2.1.
To unsubscribe from this branch go to 
https://code.launchpad.net/~mailman-coders/mailman/2.1/+edit-subscription
=== modified file 'Mailman/Message.py'
--- Mailman/Message.py	2012-02-18 02:18:34 +0000
+++ Mailman/Message.py	2014-04-27 04:25:34 +0000
@@ -1,4 +1,4 @@
-# Copyright (C) 1998-2012 by the Free Software Foundation, Inc.
+# Copyright (C) 1998-2014 by the Free Software Foundation, Inc.
 #
 # This program is free software; you can redistribute it and/or
 # modify it under the terms of the GNU General Public License
@@ -284,8 +284,13 @@
         # UserNotifications are typically for admin messages, and for messages
         # other than list explosions.  Send these out as Precedence: bulk, but
         # don't override an existing Precedence: header.
+        # Also, if the message is To: the list-owner address, set Precedence:
+        # list.  See note below in OwnerNotification.
         if not (self.has_key('precedence') or noprecedence):
-            self['Precedence'] = 'bulk'
+            if self.get('to') == mlist.GetOwnerEmail():
+                self['Precedence'] = 'list'
+            else:
+                self['Precedence'] = 'bulk'
         self._enqueue(mlist, **_kws)
 
     def _enqueue(self, mlist, **_kws):
@@ -318,6 +323,12 @@
         del self['to']
         self['To'] = mlist.GetOwnerEmail()
         self._sender = sender
+        # User notifications are normally sent with Precedence: bulk.  This
+        # is appropriate as they can be backscatter of rejected spam.
+        # Owner notifications are not backscatter and are perhaps more
+        # important than 'bulk' so give them Precedence: list by default.
+        # (LP: #1313146)
+        self['Precedence'] = 'list'
 
     def _enqueue(self, mlist, **_kws):
         # Not imported at module scope to avoid import loop

=== modified file 'NEWS'
--- NEWS	2014-04-26 05:15:07 +0000
+++ NEWS	2014-04-27 04:25:34 +0000
@@ -5,6 +5,13 @@
 
 Here is a history of user visible changes to Mailman.
 
+2.1.18 (xx-xxx-xxxx)
+
+  Bug fixes and other patches
+
+    - Most Mailman generated notices to list owners and moderators are now
+      sent as Precedence: list instead of bulk.  (LP: #1313146)
+
 2.1.18rc3 (25-Apr-2014)
 
   Bug fixes and other patches

_______________________________________________
Mailman-checkins mailing list
Mailman-checkins@python.org
Unsubscribe: 
https://mail.python.org/mailman/options/mailman-checkins/archive%40jab.org

Reply via email to