Ticket #16705: 16705.patch

django/test/client.py

@@ -182 +182 @@
         """
         The base environment for a request.
         """
+        # This is a minimal valid WSGI environ dictionary, plus HTTP_COOKIE
+        # for cookie support. Empty strings are omitted.
+        # See http://www.python.org/dev/peps/pep-3333/#environ-variables
         environ = {
             'HTTP_COOKIE':       self.cookies.output(header='', sep='; '),
-            'PATH_INFO':         '/',
-            'QUERY_STRING':      '',
-            'REMOTE_ADDR':       '127.0.0.1',
             'REQUEST_METHOD':    'GET',
-            'SCRIPT_NAME':       '',
             'SERVER_NAME':       'testserver',
             'SERVER_PORT':       '80',
             'SERVER_PROTOCOL':   'HTTP/1.1',
             'wsgi.version':      (1,0),
             'wsgi.url_scheme':   'http',
+            'wsgi.input':        FakePayload(''),
             'wsgi.errors':       self.errors,
             'wsgi.multiprocess': True,
             'wsgi.multithread':  False,
@@ -235 +235 @@
             'PATH_INFO':       self._get_path(parsed),
             'QUERY_STRING':    urlencode(data, doseq=True) or parsed[4],
             'REQUEST_METHOD': 'GET',
-            'wsgi.input':      FakePayload('')
         }
         r.update(extra)
         return self.request(**r)
@@ -267 +266 @@
             'PATH_INFO':       self._get_path(parsed),
             'QUERY_STRING':    urlencode(data, doseq=True) or parsed[4],
             'REQUEST_METHOD': 'HEAD',
-            'wsgi.input':      FakePayload('')
         }
         r.update(extra)
         return self.request(**r)
@@ -280 +278 @@
             'PATH_INFO':       self._get_path(parsed),
             'QUERY_STRING':    urlencode(data, doseq=True) or parsed[4],
             'REQUEST_METHOD': 'OPTIONS',
-            'wsgi.input':      FakePayload('')
         }
         r.update(extra)
         return self.request(**r)
@@ -311 +308 @@
             'PATH_INFO':       self._get_path(parsed),
             'QUERY_STRING':    urlencode(data, doseq=True) or parsed[4],
             'REQUEST_METHOD': 'DELETE',
-            'wsgi.input':      FakePayload('')
         }
         r.update(extra)
         return self.request(**r)

django/views/generic/simple.py

@@ -48 +48 @@
     from the request is appended to the URL.
 
     """
-    args = request.META["QUERY_STRING"]
+    args = request.META.get('QUERY_STRING', '')
     if args and query_string and url is not None:
         url = "%s?%s" % (url, args)
 

django/views/generic/base.py

@@ -140 +140 @@
         are provided as kwargs to this method.
         """
         if self.url:
-            args = self.request.META["QUERY_STRING"]
+            args = self.request.META.get('QUERY_STRING', '')
             if args and self.query_string:
                 url = "%s?%s" % (self.url, args)
             else:

django/middleware/common.py

@@ -85 +85 @@
                 new_url[0], urlquote(new_url[1]))
         else:
             newurl = urlquote(new_url[1])
-        if request.GET:
+        if request.META.get('QUERY_STRING', ''):
             newurl += '?' + request.META['QUERY_STRING']
         return http.HttpResponsePermanentRedirect(newurl)
 

tests/regressiontests/views/tests/generic/simple.py

@@ -47 +47 @@
         response = self.client.get('/simple/redirect_to_query/?param1=foo&param2=bar')
         self.assertEqual(response.status_code, 301)
         self.assertEqual('http://testserver/simple/target/?param1=foo&param2=bar', response['Location'])
+
+    def test_redirect_to_when_meta_contains_no_query_string(self):
+        "regression for #16705"
+        # we can't use self.client.get because it always sets QUERY_STRING
+        response = self.client.request(PATH_INFO='/simple/redirect_to/')
+        self.assertEqual(response.status_code, 301)
+        

tests/regressiontests/generic_views/base.py

@@ -248 +248 @@
         response = RedirectView.as_view()(self.rf.get('/foo/'))
         self.assertEqual(response.status_code, 410)
 
-    def test_permanaent_redirect(self):
+    def test_permanent_redirect(self):
         "Default is a permanent redirect"
         response = RedirectView.as_view(url='/bar/')(self.rf.get('/foo/'))
         self.assertEqual(response.status_code, 301)
@@ -305 +305 @@
         response = RedirectView.as_view(url='/bar/')(self.rf.delete('/foo/'))
         self.assertEqual(response.status_code, 301)
         self.assertEqual(response['Location'], '/bar/')
+
+    def test_redirect_when_meta_contains_no_query_string(self):
+        "regression for #16705"
+        # we can't use self.rf.get because it always sets QUERY_STRING
+        response = RedirectView.as_view(url='/bar/')(self.rf.request(PATH_INFO='/foo/'))
+        self.assertEqual(response.status_code, 301)