diff options
author | Mark Sapiro <mark@msapiro.net> | 2015-08-13 19:05:52 -0700 |
---|---|---|
committer | Mark Sapiro <mark@msapiro.net> | 2015-08-13 19:05:52 -0700 |
commit | 7fc34196cae19d5e719f6ae94ffb39debc87cf3c (patch) | |
tree | 63e7ba409a40f86ed243f4ea41ec0f6edbe780ab /Mailman | |
parent | fc381dd41d2d4db6b4e04014a4802befda767fd8 (diff) | |
download | mailman2-7fc34196cae19d5e719f6ae94ffb39debc87cf3c.tar.gz mailman2-7fc34196cae19d5e719f6ae94ffb39debc87cf3c.tar.xz mailman2-7fc34196cae19d5e719f6ae94ffb39debc87cf3c.zip |
Improved fix for LP: 1482940.
Diffstat (limited to 'Mailman')
-rw-r--r-- | Mailman/Queue/BounceRunner.py | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/Mailman/Queue/BounceRunner.py b/Mailman/Queue/BounceRunner.py index d418ac5d..651039d6 100644 --- a/Mailman/Queue/BounceRunner.py +++ b/Mailman/Queue/BounceRunner.py @@ -30,6 +30,7 @@ from Mailman import mm_cfg from Mailman import Utils from Mailman import LockFile from Mailman.Message import UserNotification +from Mailman.Bouncer import _BounceInfo from Mailman.Bouncers import BouncerAPI from Mailman.Queue.Runner import Runner from Mailman.Queue.sbcache import get_switchboard @@ -152,16 +153,17 @@ class BounceMixin: op, addr, bmsg = mlist.pend_confirm(token) info = mlist.getBounceInfo(addr) 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() + # info was deleted before probe bounce was received. + # Just create a new info. + info = _BounceInfo(addr, + 0.0, + time.localtime()[:3], + mlist.bounce_you_are_disabled_warnings + ) + 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() |