Merge pull request #1754 from Lukasa/verb_change

Match the HTTPbis on HTTP 301.
2026-06-05 22:50:18 +00:00 · 2013-11-25 11:40:00 -08:00
parent 3327652623 3369d87da9
commit e5b9ae1f8e
1 changed files with 7 additions and 2 deletions
@@ -112,8 +112,13 @@ class SessionRedirectMixin(object):
                method = 'GET'

            # Do what the browsers do, despite standards...
-            if (resp.status_code in (codes.moved, codes.found) and
-                    method not in ('GET', 'HEAD')):
+            # First, turn 302s into GETs.
+            if resp.status_code == codes.found and method != 'HEAD':
+                method = 'GET'
+
+            # Second, if a POST is responded to with a 301, turn it into a GET.
+            # This bizarre behaviour is explained in Issue 1704.
+            if resp.status_code == codes.moved and method == 'POST':
                method = 'GET'

            prepared_request.method = method