aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMark Sapiro <mark@msapiro.net>2010-01-21 20:39:56 -0800
committerMark Sapiro <mark@msapiro.net>2010-01-21 20:39:56 -0800
commit7b9209cab3c1f08f1f594399aa140b729ad70760 (patch)
tree9b06aef8621bdd6839ae5f5060e7608ea082c79d
parent034ffbb9dc7dbee5c7e9ed73ebf346a04252b4bd (diff)
downloadmailman2-7b9209cab3c1f08f1f594399aa140b729ad70760.tar.gz
mailman2-7b9209cab3c1f08f1f594399aa140b729ad70760.tar.xz
mailman2-7b9209cab3c1f08f1f594399aa140b729ad70760.zip
- 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.
-rw-r--r--Mailman/Cgi/private.py4
-rw-r--r--NEWS3
2 files changed, 6 insertions, 1 deletions
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.