------------------------------------------------------------
revno: 1571
fixes bug: https://launchpad.net/bugs/1482940
committer: Mark Sapiro <m...@msapiro.net>
branch nick: 2.1
timestamp: Sat 2015-08-08 20:51:59 -0700
message:
  Fixed bug with delayed probe bounces.
modified:
  Mailman/Queue/BounceRunner.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/Queue/BounceRunner.py'
--- Mailman/Queue/BounceRunner.py	2013-03-28 22:12:45 +0000
+++ Mailman/Queue/BounceRunner.py	2015-08-09 03:51:59 +0000
@@ -1,4 +1,4 @@
-# Copyright (C) 2001-2013 by the Free Software Foundation, Inc.
+# Copyright (C) 2001-2015 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
@@ -151,10 +151,17 @@
         try:
             op, addr, bmsg = mlist.pend_confirm(token)
             info = mlist.getBounceInfo(addr)
-            mlist.disableBouncingMember(addr, info, bmsg)
-            # Only save the list if we're unlocking it
-            if not locked:
-                mlist.Save()
+            if not info:
+                syslog('bounce',
+                       '%s: Probe bounce received for %s with no bounce info',
+                       mlist.internal_name(),
+                       addr)
+                maybe_forward(mlist, bmsg)
+            else:
+                mlist.disableBouncingMember(addr, info, bmsg)
+                # Only save the list if we're unlocking it
+                if not locked:
+                    mlist.Save()
         finally:
             if not locked:
                 mlist.Unlock()

=== modified file 'NEWS'
--- NEWS	2015-07-25 15:27:52 +0000
+++ NEWS	2015-08-09 03:51:59 +0000
@@ -14,6 +14,10 @@
 
   Bug fixes and other patches
 
+    - Fixed a bug where a delayed probe bounce can throw an AttributeError.
+      Now this will be logged and the probe bounce forwarded to the list
+      admin if bounce_unrecognized_goes_to_list_owner is Yes.  (LP: #1482940)
+
     - If a list is not digestable an the user is not currently set to
       receive digests, the digest options will not be shown on the user's
       options page.  (LP: #1476298)

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

Reply via email to