fixed bug that causes improper redirect logic after POST

Author: Athlon1600

src/Client.php

@@ -45,11 +45,16 @@ public function request($method, $uri, $params = array(), $headers = array(), $c
             curl_setopt($ch, CURLOPT_URL, $uri);
 
             $post_data = is_array($params) ? http_build_query($params) : $params;
+
+            if ($method == 'POST') {
+                curl_setopt($ch, CURLOPT_POST, 1);
+            } else {
+                curl_setopt($ch, CURLOPT_CUSTOMREQUEST, $method);
+            }
+
             curl_setopt($ch, CURLOPT_POSTFIELDS, $post_data);
         }
 
-        curl_setopt($ch, CURLOPT_CUSTOMREQUEST, $method);
-
         curl_setopt($ch, CURLOPT_HTTPHEADER, $this->getCombinedHeaders($headers));
         curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
         curl_setopt($ch, CURLOPT_HEADER, 0);

tests/HttpBinTest.php

@@ -83,4 +83,17 @@ public function test_redirect_nofollow()
 
         $this->assertEquals(302, $response->status);
     }
+
+    // https://github.com/Kong/insomnia/issues/227
+    public function test_redirect_switch_to_get()
+    {
+        $client = new Client();
+
+        $response = $client->post('https://httpbin.org/redirect-to?url=https://www.google.com/&status_code=301');
+
+        // if CURLOPT_CUSTOMREQUEST was set, then redirect would follow from POST -> POST, rather than POST -> GET
+        // POST to google index gives 405
+        //  <p>The request method <code>POST</code> is inappropriate for the URL <code>/</code>.  <ins>That’s all we know.</ins>
+        $this->assertEquals(200, $response->status);
+    }
 }