From 5e2f50ff61060fd11ae19ea43808f4a6b07d5b0a Mon Sep 17 00:00:00 2001 From: Mark Sapiro Date: Tue, 29 Mar 2011 20:33:24 -0700 Subject: Changed the subscribe CGI to strip leading and trailing whitespace from the supplied email address. Bug #745432. --- Mailman/Cgi/subscribe.py | 4 ++-- NEWS | 3 +++ 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. -- cgit v1.2.3