changeset:   98509:969afbf501af
branch:      3.4
parent:      98505:bb1a8d3dd4a1
user:        Martin Panter <vadmium+py@gmail.com>
date:        Sat Oct 03 05:38:07 2015 +0000
files:       Lib/http/server.py Lib/test/test_httpservers.py Misc/ACKS Misc/NEWS
description:
Issue #25232: Fix CGIRequestHandler's splitting of URL query

Patch from Xiang Zhang.


diff -r bb1a8d3dd4a1 -r 969afbf501af Lib/http/server.py
--- a/Lib/http/server.py	Fri Oct 02 23:22:59 2015 -0400
+++ b/Lib/http/server.py	Sat Oct 03 05:38:07 2015 +0000
@@ -1032,11 +1032,7 @@
                 break
 
         # find an explicit query string, if present.
-        i = rest.rfind('?')
-        if i >= 0:
-            rest, query = rest[:i], rest[i+1:]
-        else:
-            query = ''
+        rest, _, query = rest.partition('?')
 
         # dissect the part after the directory name into a script name &
         # a possible additional path, to be stored in PATH_INFO.
diff -r bb1a8d3dd4a1 -r 969afbf501af Lib/test/test_httpservers.py
--- a/Lib/test/test_httpservers.py	Fri Oct 02 23:22:59 2015 -0400
+++ b/Lib/test/test_httpservers.py	Sat Oct 03 05:38:07 2015 +0000
@@ -366,6 +366,16 @@
                           form.getfirst("bacon")))
 """
 
+cgi_file4 = """\
+#!%s
+import os
+
+print("Content-type: text/html")
+print()
+
+print(os.environ["%s"])
+"""
+
 
 @unittest.skipIf(hasattr(os, 'geteuid') and os.geteuid() == 0,
         "This test can't be run reliably as root (issue #13308).")
@@ -387,6 +397,7 @@
         self.file1_path = None
         self.file2_path = None
         self.file3_path = None
+        self.file4_path = None
 
         # The shebang line should be pure ASCII: use symlink if possible.
         # See issue #7668.
@@ -425,6 +436,11 @@
             file3.write(cgi_file1 % self.pythonexe)
         os.chmod(self.file3_path, 0o777)
 
+        self.file4_path = os.path.join(self.cgi_dir, 'file4.py')
+        with open(self.file4_path, 'w', encoding='utf-8') as file4:
+            file4.write(cgi_file4 % (self.pythonexe, 'QUERY_STRING'))
+        os.chmod(self.file4_path, 0o777)
+
         os.chdir(self.parent_dir)
 
     def tearDown(self):
@@ -440,6 +456,8 @@
                 os.remove(self.file2_path)
             if self.file3_path:
                 os.remove(self.file3_path)
+            if self.file4_path:
+                os.remove(self.file4_path)
             os.rmdir(self.cgi_child_dir)
             os.rmdir(self.cgi_dir)
             os.rmdir(self.parent_dir)
@@ -541,6 +559,12 @@
         self.assertEqual((b'Hello World' + self.linesep, 'text/html', 200),
                 (res.read(), res.getheader('Content-type'), res.status))
 
+    def test_query_with_multiple_question_mark(self):
+        res = self.request('/cgi-bin/file4.py?a=b?c=d')
+        self.assertEqual(
+            (b'a=b?c=d' + self.linesep, 'text/html', 200),
+            (res.read(), res.getheader('Content-type'), res.status))
+
 
 class SocketlessRequestHandler(SimpleHTTPRequestHandler):
     def __init__(self):
diff -r bb1a8d3dd4a1 -r 969afbf501af Misc/ACKS
--- a/Misc/ACKS	Fri Oct 02 23:22:59 2015 -0400
+++ b/Misc/ACKS	Sat Oct 03 05:38:07 2015 +0000
@@ -1531,6 +1531,7 @@
 Daniel Wozniak
 Heiko Wundram
 Doug Wyatt
+Xiang Zhang
 Robert Xiao
 Florent Xicluna
 Hirokazu Yamamoto
diff -r bb1a8d3dd4a1 -r 969afbf501af Misc/NEWS
--- a/Misc/NEWS	Fri Oct 02 23:22:59 2015 -0400
+++ b/Misc/NEWS	Sat Oct 03 05:38:07 2015 +0000
@@ -90,6 +90,9 @@
 Library
 -------
 
+- Issue #25232: Fix CGIRequestHandler to split the query from the URL at the
+  first question mark (?) rather than the last. Patch from Xiang Zhang.
+
 - Issue #22958: Constructor and update method of weakref.WeakValueDictionary
   now accept the self and the dict keyword arguments.