[HttpClient] Don't send any default content-type when the body is empty

nicolas-grekas · nicolas-grekas · commit 3294a433fc9d · 2025-02-13T10:55:13.000+01:00
diff --git a/HttpClientTrait.php b/HttpClientTrait.php
@@ -356,9 +356,11 @@ private static function normalizeBody($body, array &$normalizedHeaders = [])
                 }
             });
 
-            $body = http_build_query($body, '', '&');
+            if ('' === $body = http_build_query($body, '', '&')) {
+                return '';
+            }
 
-            if ('' === $body || !$streams && !str_contains($normalizedHeaders['content-type'][0] ?? '', 'multipart/form-data')) {
+            if (!$streams && !str_contains($normalizedHeaders['content-type'][0] ?? '', 'multipart/form-data')) {
                 if (!str_contains($normalizedHeaders['content-type'][0] ?? '', 'application/x-www-form-urlencoded')) {
                     $normalizedHeaders['content-type'] = ['Content-Type: application/x-www-form-urlencoded'];
                 }

Original file line number	Diff line number	Diff line change
`@@ -356,9 +356,11 @@ private static function normalizeBody($body, array &$normalizedHeaders = [])`
`356`	`356`	`}`
`357`	`357`	`});`
`358`	`358`
`359`		`- $body = http_build_query($body, '', '&');`
	`359`	`+ if ('' === $body = http_build_query($body, '', '&')) {`
	`360`	`+ return '';`
	`361`	`+ }`
`360`	`362`
`361`		`- if ('' === $body \|\| !$streams && !str_contains($normalizedHeaders['content-type'][0] ?? '', 'multipart/form-data')) {`
	`363`	`+ if (!$streams && !str_contains($normalizedHeaders['content-type'][0] ?? '', 'multipart/form-data')) {`
`362`	`364`	`if (!str_contains($normalizedHeaders['content-type'][0] ?? '', 'application/x-www-form-urlencoded')) {`
`363`	`365`	`$normalizedHeaders['content-type'] = ['Content-Type: application/x-www-form-urlencoded'];`
`364`	`366`	`}`