aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--Mailman/SecurityManager.py5
-rw-r--r--NEWS4
2 files changed, 7 insertions, 2 deletions
diff --git a/Mailman/SecurityManager.py b/Mailman/SecurityManager.py
index 01610b43..572018e2 100644
--- a/Mailman/SecurityManager.py
+++ b/Mailman/SecurityManager.py
@@ -1,4 +1,4 @@
-# Copyright (C) 1998-2006 by the Free Software Foundation, Inc.
+# Copyright (C) 1998-2008 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
@@ -299,7 +299,8 @@ class SecurityManager:
usernames.append(k[len(prefix):])
# If any check out, we're golden. Note: `@'s are no longer legal
# values in cookie keys.
- for user in [Utils.UnobscureEmail(u) for u in usernames]:
+ for user in [Utils.UnobscureEmail(urllib.unquote(u))
+ for u in usernames]:
ok = self.__checkone(c, authcontext, user)
if ok:
return True
diff --git a/NEWS b/NEWS
index b58b8eef..91418076 100644
--- a/NEWS
+++ b/NEWS
@@ -31,6 +31,10 @@ Here is a history of user visible changes to Mailman.
in the To: or Cc: header differed in case from the case-preserved member
address. Bug #297795.
+ - Fixed a problem in SecurityManager that caused it to not find the
+ cookie when CheckCookie was not given a user and the user in the cookie
+ had a %xx encoded character. Bug # 299220.
+
2.1.11 (30-Jun-2008)
New Features