diff options
| author | Mark Sapiro <mark@msapiro.net> | 2008-06-22 12:28:05 -0700 |
|---|---|---|
| committer | Mark Sapiro <mark@msapiro.net> | 2008-06-22 12:28:05 -0700 |
| commit | 7ad9af1703e5762cc7cb4d2d02303975b2e6f67a (patch) | |
| tree | dffa7de7f997fb144e8921de97fdc7d04147b996 /Mailman/Queue | |
| parent | 9f3b515a2fae45b80449e2cde8378628a2e9c705 (diff) | |
| download | mailman2-7ad9af1703e5762cc7cb4d2d02303975b2e6f67a.tar.gz mailman2-7ad9af1703e5762cc7cb4d2d02303975b2e6f67a.tar.xz mailman2-7ad9af1703e5762cc7cb4d2d02303975b2e6f67a.zip | |
Another BounceRunner loop detection change (rev 1106 wasn't quite right).
Diffstat (limited to 'Mailman/Queue')
| -rw-r--r-- | Mailman/Queue/BounceRunner.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/Mailman/Queue/BounceRunner.py b/Mailman/Queue/BounceRunner.py index 6405c091..d219d6e9 100644 --- a/Mailman/Queue/BounceRunner.py +++ b/Mailman/Queue/BounceRunner.py @@ -209,8 +209,10 @@ class BounceRunner(Runner, BounceMixin): return # Is this a possible looping message sent directly to a list-bounces # address other than the site list? - # Use the From: because message may not have a unix_from. - if msg.get('from') == Utils.get_site_email(extra='bounces'): + # Check From: because unix_from might be VERP'd. + # Also, check the From: that Message.OwnerNotification uses. + if (msg.get('from') == + Utils.get_site_email(mlist.host_name, 'bounces')): # Just send it to the sitelist-owner address. If that bounces # we'll handle it above. outq.enqueue(msg, msgdata, |