ensure existing content-length headers are not overwritten and ignore negative results from stream getSize in HttplugClient and Psr18Client

KurtThiemann · KurtThiemann · commit 5ed994bf28d2 · 2024-11-13T19:25:58.000+01:00
diff --git a/src/Symfony/Component/HttpClient/HttplugClient.php b/src/Symfony/Component/HttpClient/HttplugClient.php
@@ -230,7 +230,7 @@ private function sendPsr7Request(RequestInterface $request, ?bool $buffer = null
             }
 
             $headers = $request->getHeaders();
-            if (($size = $body->getSize()) !== null) {
+            if (!$request->hasHeader('content-length') && 0 <= $size = $body->getSize() ?? -1) {
                 $headers['Content-Length'] = [$size];
             }
 
diff --git a/src/Symfony/Component/HttpClient/Psr18Client.php b/src/Symfony/Component/HttpClient/Psr18Client.php
@@ -94,7 +94,7 @@ public function sendRequest(RequestInterface $request): ResponseInterface
             }
 
             $headers = $request->getHeaders();
-            if (($size = $body->getSize()) !== null) {
+            if (!$request->hasHeader('content-length') && 0 <= $size = $body->getSize() ?? -1) {
                 $headers['Content-Length'] = [$size];
             }
 
