------------------------------------------------------------
revno: 1244
committer: Mark Sapiro <msap...@value.net>
branch nick: 2.2
timestamp: Thu 2013-03-28 14:22:28 -0700
message:
  Fixed a bug where BounceRunner could create and leave behind zero length
  bounce-events files.  (LP: 1161610)
modified:
  Mailman/Queue/BounceRunner.py
  NEWS


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

Your team Mailman Checkins is subscribed to branch lp:mailman/2.2.
To unsubscribe from this branch go to 
https://code.launchpad.net/~mailman-coders/mailman/2.2/+edit-subscription
=== modified file 'Mailman/Queue/BounceRunner.py'
--- Mailman/Queue/BounceRunner.py	2008-06-22 19:28:05 +0000
+++ Mailman/Queue/BounceRunner.py	2013-03-28 21:22:28 +0000
@@ -244,6 +244,7 @@
                 return
         # If that still didn't return us any useful addresses, then send it on
         # or discard it.
+        addrs = filter(None, addrs)
         if not addrs:
             syslog('bounce',
                    '%s: bounce message w/no discernable addresses: %s',
@@ -254,7 +255,8 @@
         # BAW: It's possible that there are None's in the list of addresses,
         # although I'm unsure how that could happen.  Possibly ScanMessages()
         # can let None's sneak through.  In any event, this will kill them.
-        addrs = filter(None, addrs)
+        # addrs = filter(None, addrs)
+        # MAS above filter moved up so we don't try to queue an empty list.
         self._queue_bounces(mlist.internal_name(), addrs, msg)
 
     _doperiodic = BounceMixin._doperiodic

=== modified file 'NEWS'
--- NEWS	2013-03-26 02:02:48 +0000
+++ NEWS	2013-03-28 21:22:28 +0000
@@ -102,6 +102,9 @@
 
   Bug Fixes and other patches
 
+    - Fixed a bug where BounceRunner could create and leave behind zero length
+      bounce-events files.  (LP: 1161610)
+
     - Added recognition for another Yahoo bounce format.  LP: #1157961
     - Changed configure's method for getting Python's include directory from
       distutils.sysconfig.get_config_var('CONFINCLUDEPY') to

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

Reply via email to