aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMark Sapiro <mark@msapiro.net>2014-12-07 20:51:42 -0800
committerMark Sapiro <mark@msapiro.net>2014-12-07 20:51:42 -0800
commitcb8225458e7044a4f096bbbb6fb5a3126d691f41 (patch)
tree7e8d8845b495b3bba4b763ef7074cd5362bc742e
parent90cf692002ba35728385ee0bfacab578944bdd82 (diff)
downloadmailman2-cb8225458e7044a4f096bbbb6fb5a3126d691f41.tar.gz
mailman2-cb8225458e7044a4f096bbbb6fb5a3126d691f41.tar.xz
mailman2-cb8225458e7044a4f096bbbb6fb5a3126d691f41.zip
Accept email command in Subject: prefixed with Re: or similar with no
intervening space.
Diffstat (limited to '')
-rw-r--r--Mailman/Queue/CommandRunner.py12
-rwxr-xr-xNEWS3
2 files changed, 13 insertions, 2 deletions
diff --git a/Mailman/Queue/CommandRunner.py b/Mailman/Queue/CommandRunner.py
index b63b050c..d25fe87e 100644
--- a/Mailman/Queue/CommandRunner.py
+++ b/Mailman/Queue/CommandRunner.py
@@ -1,4 +1,4 @@
-# Copyright (C) 1998-2011 by the Free Software Foundation, Inc.
+# Copyright (C) 1998-2014 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
@@ -138,9 +138,17 @@ class Results:
# localized version thereof) on the Subject: line that's messing
# things up. Pop the prefix off and try again... once.
#
+ # At least one MUA (163.com web mail) has been observed that
+ # inserts 'Re:' with no following space, so try to account for
+ # that too.
+ #
# If that still didn't work it isn't enough to stop processing.
# BAW: should we include a message that the Subject: was ignored?
- if not self.subjcmdretried and args:
+ if self.subjcmdretried < 1:
+ self.subjcmdretried += 1
+ if re.search('^.*:.+', cmd):
+ return self.do_command(re.sub('.*:', '', cmd), args)
+ if self.subjcmdretried < 2 and args:
self.subjcmdretried += 1
cmd = args.pop(0)
return self.do_command(cmd, args)
diff --git a/NEWS b/NEWS
index 11795058..d2013f40 100755
--- a/NEWS
+++ b/NEWS
@@ -33,6 +33,9 @@ Here is a history of user visible changes to Mailman.
Bug fixes and other patches
+ - Accept email command in Subject: prefixed with Re: or similar with no
+ intervening space. (LP: #1400200)
+
- Fixed a UnicodeDecodeError that could occur in the web admin interface
if 'text' valued attributes have unicode values. (LP: #1397170)