aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorbwarsaw <>2004-01-03 01:13:26 +0000
committerbwarsaw <>2004-01-03 01:13:26 +0000
commitf17ee4947a4e25cc4e94b96b1142764e0c856d2d (patch)
tree86dc7e105b30dd34966a90ef18d552fd40dbf0db
parente47ce33c948c80ee641399ac1c513546e2a37031 (diff)
downloadmailman2-f17ee4947a4e25cc4e94b96b1142764e0c856d2d.tar.gz
mailman2-f17ee4947a4e25cc4e94b96b1142764e0c856d2d.tar.xz
mailman2-f17ee4947a4e25cc4e94b96b1142764e0c856d2d.zip
__checkone(): Patch 869644, ignore NotAMemberError that can get raised
from AuthContextInfo() when old cookie data is for someone no longer a member of the mailing list. Closes SF bug # 869647. Bug and patch by Stephan Berndts.
Diffstat (limited to '')
-rw-r--r--Mailman/SecurityManager.py7
1 files changed, 5 insertions, 2 deletions
diff --git a/Mailman/SecurityManager.py b/Mailman/SecurityManager.py
index 62360feb..4ebb1fcd 100644
--- a/Mailman/SecurityManager.py
+++ b/Mailman/SecurityManager.py
@@ -1,4 +1,4 @@
-# Copyright (C) 1998-2003 by the Free Software Foundation, Inc.
+# Copyright (C) 1998-2004 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
@@ -307,7 +307,10 @@ class SecurityManager:
def __checkone(self, c, authcontext, user):
# Do the guts of the cookie check, for one authcontext/user
# combination.
- key, secret = self.AuthContextInfo(authcontext, user)
+ try:
+ key, secret = self.AuthContextInfo(authcontext, user)
+ except Errors.NotAMemberError:
+ return False
if not c.has_key(key) or not isinstance(secret, StringType):
return False
# Undo the encoding we performed in MakeCookie() above. BAW: I