aboutsummaryrefslogtreecommitdiffstats
path: root/Mailman
diff options
context:
space:
mode:
authorMark Sapiro <mark@msapiro.net>2015-08-13 19:05:52 -0700
committerMark Sapiro <mark@msapiro.net>2015-08-13 19:05:52 -0700
commit7fc34196cae19d5e719f6ae94ffb39debc87cf3c (patch)
tree63e7ba409a40f86ed243f4ea41ec0f6edbe780ab /Mailman
parentfc381dd41d2d4db6b4e04014a4802befda767fd8 (diff)
downloadmailman2-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.py22
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()