aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMark Sapiro <msapiro@value.net>2011-03-29 20:33:24 -0700
committerMark Sapiro <msapiro@value.net>2011-03-29 20:33:24 -0700
commit5e2f50ff61060fd11ae19ea43808f4a6b07d5b0a (patch)
treeafb08df34f3e57cd19d53d47115752a00fd51cad
parent07a595eea998d7b71f74ae6b5acf3d3870a5cd31 (diff)
downloadmailman2-5e2f50ff61060fd11ae19ea43808f4a6b07d5b0a.tar.gz
mailman2-5e2f50ff61060fd11ae19ea43808f4a6b07d5b0a.tar.xz
mailman2-5e2f50ff61060fd11ae19ea43808f4a6b07d5b0a.zip
Changed the subscribe CGI to strip leading and trailing whitespace from
the supplied email address. Bug #745432.
Diffstat (limited to '')
-rw-r--r--Mailman/Cgi/subscribe.py4
-rw-r--r--NEWS3
2 files changed, 5 insertions, 2 deletions
diff --git a/Mailman/Cgi/subscribe.py b/Mailman/Cgi/subscribe.py
index f7b03ac6..0d10eb19 100644
--- a/Mailman/Cgi/subscribe.py
+++ b/Mailman/Cgi/subscribe.py
@@ -1,4 +1,4 @@
-# Copyright (C) 1998-2009 by the Free Software Foundation, Inc.
+# Copyright (C) 1998-2011 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
@@ -109,7 +109,7 @@ def process_form(mlist, doc, cgidata, lang):
results = []
# The email address being subscribed, required
- email = cgidata.getvalue('email', '')
+ email = cgidata.getvalue('email', '').strip()
if not email:
results.append(_('You must supply a valid email address.'))
diff --git a/NEWS b/NEWS
index 3e297cf4..5ec7c3f9 100644
--- a/NEWS
+++ b/NEWS
@@ -38,6 +38,9 @@ Here is a history of user visible changes to Mailman.
Bug Fixes and other patches
+ - Changed the subscribe CGI to strip leading and trailing whitespace from
+ the supplied email address. Bug #745432.
+
- Changed the maximum number of arguments for the who command to be
considered administrivia from 2 to 1 to help avoid false positives.
Bug #739524.