From 3cf944a0c4457ef5b10c12f34327645002995a7b Mon Sep 17 00:00:00 2001 From: Mark Sapiro Date: Sun, 13 Sep 2009 15:52:21 -0700 Subject: Fixed the auto-responder to treat messages to -confirm, -join, -leave, -subscribe and -unsubscribe as requests rather than posts. Bug #427962. --- Mailman/Handlers/Replybot.py | 9 +++++---- NEWS | 3 +++ 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/Mailman/Handlers/Replybot.py b/Mailman/Handlers/Replybot.py index 6cd74ca8..3c9536a0 100644 --- a/Mailman/Handlers/Replybot.py +++ b/Mailman/Handlers/Replybot.py @@ -43,11 +43,12 @@ def process(mlist, msg, msgdata): if ack <> 'yes' and precedence in ('bulk', 'junk', 'list'): return # Check to see if the list is even configured to autorespond to this email - # message. Note: the mailowner script sets the `toadmin' or `toowner' key - # (which for replybot purposes are equivalent), and the mailcmd script - # sets the `torequest' key. + # message. Note: the owner script sets the `toowner' key, and the various + # confirm, join, leave, request, subscribe and unsubscribe scripts set the + # keys we use for `torequest'. toadmin = msgdata.get('toowner') - torequest = msgdata.get('torequest') + torequest = msgdata.get('torequest') or msgdata.get('toconfirm') or \ + msgdata.get('tojoin') or msgdata.get('toleave') if ((toadmin and not mlist.autorespond_admin) or (torequest and not mlist.autorespond_requests) or \ (not toadmin and not torequest and not mlist.autorespond_postings)): diff --git a/NEWS b/NEWS index ef848f82..4f14a389 100644 --- a/NEWS +++ b/NEWS @@ -8,6 +8,9 @@ Here is a history of user visible changes to Mailman. Bug Fixes and other patches + - Fixed the auto-responder to treat messages to -confirm, -join, -leave, + -subscribe and -unsubscribe as requests rather than posts. Bug #427962. + - Configure/make no longer builds Japanese and Korean codecs in pythonlib if Python already has them. -- cgit v1.2.3