[9.x] Skip parameter parsing for raw post body in HTTP Client (larave…

…l#42364) * Skip parameter parsing for raw post body in HTTP Client * Reduce test string length
chu121su12 · May 14, 2022 · 539f0ed · 539f0ed
1 parent c5acd1a
commit 539f0ed
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/src/Illuminate/Http/Client/PendingRequest.php b/src/Illuminate/Http/Client/PendingRequest.php
@@ -932,6 +932,10 @@ protected function sendRequest(/*string */$method, /*string */$url, array $optio
      */
     protected function parseRequestData($method, $url, array $options)
     {
+        if ($this->bodyFormat === 'body') {
+            return [];
+        }
+
         $laravelData = isset($options[$this->bodyFormat])
             ? $options[$this->bodyFormat]
             : (isset($options['query']) ? $options['query'] : []);

diff --git a/tests/Http/HttpClientTest.php b/tests/Http/HttpClientTest.php
@@ -187,6 +187,21 @@ public function testSendRequestBody()
         $this->factory->withBody($body, 'application/json')->send('get', 'http://foo.com/api');
     }
 
+    public function testSendRequestBodyWithManyAmpersands()
+    {
+        $body = str_repeat('A thousand &. ', 1000);
+
+        $fakeRequest = function (Request $request) use ($body) {
+            self::assertSame($body, $request->body());
+
+            return ['my' => 'response'];
+        };
+
+        $this->factory->fake($fakeRequest);
+
+        $this->factory->withBody($body, 'text/plain')->send('post', 'http://foo.com/api');
+    }
+
     public function testUrlsCanBeStubbedByPath()
     {
         $this->factory->fake([