From 7b9209cab3c1f08f1f594399aa140b729ad70760 Mon Sep 17 00:00:00 2001 From: Mark Sapiro Date: Thu, 21 Jan 2010 20:39:56 -0800 Subject: - Fixed a bug where going to an archives/private/list.mbox/list.mbox URL would result in a munged URL if authentication was required. Bug #266164. --- Mailman/Cgi/private.py | 4 +++- NEWS | 3 +++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/Mailman/Cgi/private.py b/Mailman/Cgi/private.py index 86608418..c647653b 100644 --- a/Mailman/Cgi/private.py +++ b/Mailman/Cgi/private.py @@ -1,4 +1,4 @@ -# Copyright (C) 1998-2006 by the Free Software Foundation, Inc. +# Copyright (C) 1998-2010 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 @@ -137,6 +137,8 @@ def main(): # Put the original full path in the authorization form, but avoid # trailing slash if we're not adding parts. We add it below. action = mlist.GetScriptURL('private', absolute=1) + if mboxfile: + action += '.mbox' if parts[1:]: action = os.path.join(action, SLASH.join(parts[1:])) # If we added '/index.html' to true_filename, add a slash to the URL. diff --git a/NEWS b/NEWS index 441dd342..0a96a979 100644 --- a/NEWS +++ b/NEWS @@ -13,6 +13,9 @@ Here is a history of user visible changes to Mailman. Bug Fixes and other patches + - Fixed a bug where going to an archives/private/list.mbox/list.mbox URL + would result in a munged URL if authentication was required. Bug #266164. + - Fixed a bug where check_perms would throw an OSError if an entry in Mailman's lists/ directory was not a directory. Bug #265613. -- cgit v1.2.3