no need to use so much code

mojolicious · Sep 18, 2016 · 56cef83 · 56cef83
1 parent 545da2a
commit 56cef83
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/Changes b/Changes
@@ -1,5 +1,7 @@
 
 7.07  2016-09-18
+  - Fixed bug in Mojo::UserAgent::Transactor where 303 redirects would not be
+    followed correctly with GET requests. (jberger)
 
 7.06  2016-09-17
   - Fixed bug where Mojolicious::Renderer would wrap text in layouts.

diff --git a/lib/Mojo/UserAgent/Transactor.pm b/lib/Mojo/UserAgent/Transactor.pm
@@ -89,10 +89,8 @@ sub redirect {
     $new->req($clone);
   }
   else {
-    my $method = uc $req->method;
-    my $headers
-      = $new->req->method(
-      $code == 303 ? 'GET' : $method eq 'POST' ? 'GET' : $method)
+    my $m = uc $req->method;
+    my $headers = $new->req->method($code == 303 || $m eq 'POST' ? 'GET' : $m)
       ->content->headers($req->headers->clone)->headers;
     $headers->remove($_) for grep {/^content-/i} @{$headers->names};
   }