symfony
diff --git a/‎composer.json
Copy file name to clipboardExpand all lines: composer.json
+1Lines changed: 1 addition & 0 deletions b/‎composer.json
Copy file name to clipboardExpand all lines: composer.json
+1Lines changed: 1 addition & 0 deletions
diff --git a/‎src/Symfony/Component/HttpClient/Internal/HttplugWaitLoop.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/HttpClient/Internal/HttplugWaitLoop.php
+5-1Lines changed: 5 additions & 1 deletion b/‎src/Symfony/Component/HttpClient/Internal/HttplugWaitLoop.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/HttpClient/Internal/HttplugWaitLoop.php
+5-1Lines changed: 5 additions & 1 deletion
diff --git a/‎src/Symfony/Component/HttpClient/Tests/HttplugClientTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/HttpClient/Tests/HttplugClientTest.php
+18Lines changed: 18 additions & 0 deletions b/‎src/Symfony/Component/HttpClient/Tests/HttplugClientTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/HttpClient/Tests/HttplugClientTest.php
+18Lines changed: 18 additions & 0 deletions
@@ -165,6 +165,7 @@
     },
     "config": {
         "allow-plugins": {
+            "php-http/discovery": false,
             "symfony/runtime": true
         }
     },
 
@@ -120,7 +120,11 @@ public function createPsr7Response(ResponseInterface $response, bool $buffer = f
 
         foreach ($response->getHeaders(false) as $name => $values) {
             foreach ($values as $value) {
-                $psrResponse = $psrResponse->withAddedHeader($name, $value);
+                try {
+                    $psrResponse = $psrResponse->withAddedHeader($name, $value);
+                } catch (\InvalidArgumentException $e) {
+                    // ignore invalid header
+                }
             }
         }
 
 
@@ -267,4 +267,22 @@ function (\Exception $exception) use ($errorMessage, &$failureCallableCalled, $c
         $this->assertSame(200, $response->getStatusCode());
         $this->assertSame('OK', (string) $response->getBody());
     }
+
+    public function testInvalidHeaderResponse()
+    {
+        $responseHeaders = [
+            // space in header name not allowed in RFC 7230
+            ' X-XSS-Protection' => '0',
+            'Cache-Control' => 'no-cache',
+        ];
+        $response = new MockResponse('body', ['response_headers' => $responseHeaders]);
+        $this->assertArrayHasKey(' x-xss-protection', $response->getHeaders());
+
+        $client = new HttplugClient(new MockHttpClient($response));
+        $request = $client->createRequest('POST', 'http://localhost:8057/post')
+            ->withBody($client->createStream('foo=0123456789'));
+
+        $resultResponse = $client->sendRequest($request);
+        $this->assertCount(1, $resultResponse->getHeaders());
+    }
 }
Original file line number	Diff line number	Diff line change
`@@ -165,6 +165,7 @@`
`165`	`165`	`},`
`166`	`166`	`"config": {`
`167`	`167`	`"allow-plugins": {`
	`168`	`+ "php-http/discovery": false,`
`168`	`169`	`"symfony/runtime": true`
`169`	`170`	`}`
`170`	`171`	`},`
Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,11 @@ public function createPsr7Response(ResponseInterface $response, bool $buffer = f`
`120`	`120`
`121`	`121`	`foreach ($response->getHeaders(false) as $name => $values) {`
`122`	`122`	`foreach ($values as $value) {`
`123`		`- $psrResponse = $psrResponse->withAddedHeader($name, $value);`
	`123`	`+ try {`
	`124`	`+ $psrResponse = $psrResponse->withAddedHeader($name, $value);`
	`125`	`+ } catch (\InvalidArgumentException $e) {`
	`126`	`+ // ignore invalid header`
	`127`	`+ }`
`124`	`128`	`}`
`125`	`129`	`}`
`126`	`130`