[HttpClient] fix PHP warning + accept status code >= 600

nicolas-grekas · nicolas-grekas · commit 0b2645d80d19 · 2020-05-15T13:58:21.000+02:00
diff --git a/src/Symfony/Component/HttpClient/Response/CurlResponse.php b/src/Symfony/Component/HttpClient/Response/CurlResponse.php
@@ -312,8 +312,16 @@ private static function parseHeaderLine($ch, string $data, array &$info, array &
         }
 
         if ("\r\n" !== $data) {
-            // Regular header line: add it to the list
-            self::addResponseHeaders([substr($data, 0, -2)], $info, $headers);
+            try {
+                // Regular header line: add it to the list
+                self::addResponseHeaders([substr($data, 0, -2)], $info, $headers);
+            } catch (TransportException $e) {
+                $multi->handlesActivity[$id][] = null;
+                $multi->handlesActivity[$id][] = $e;
+
+                return \strlen($data);
+            }
+
 
             if (0 !== strpos($data, 'HTTP/')) {
                 if (0 === stripos($data, 'Location:')) {
diff --git a/src/Symfony/Component/HttpClient/Response/ResponseTrait.php b/src/Symfony/Component/HttpClient/Response/ResponseTrait.php
@@ -253,7 +253,7 @@ private static function initialize(self $response): void
     private static function addResponseHeaders(array $responseHeaders, array &$info, array &$headers, string &$debug = ''): void
     {
         foreach ($responseHeaders as $h) {
-            if (11 <= \strlen($h) && '/' === $h[4] && preg_match('#^HTTP/\d+(?:\.\d+)? ([12345]\d\d)(?: |$)#', $h, $m)) {
+            if (11 <= \strlen($h) && '/' === $h[4] && preg_match('#^HTTP/\d+(?:\.\d+)? ([123456789]\d\d)(?: |$)#', $h, $m)) {
                 if ($headers) {
                     $debug .= "< \r\n";
                     $headers = [];

Original file line number	Diff line number	Diff line change
`@@ -253,7 +253,7 @@ private static function initialize(self $response): void`
`253`	`253`	`private static function addResponseHeaders(array $responseHeaders, array &$info, array &$headers, string &$debug = ''): void`
`254`	`254`	`{`
`255`	`255`	`foreach ($responseHeaders as $h) {`
`256`		`- if (11 <= \strlen($h) && '/' === $h[4] && preg_match('#^HTTP/\d+(?:\.\d+)? ([12345]\d\d)(?: \|$)#', $h, $m)) {`
	`256`	`+ if (11 <= \strlen($h) && '/' === $h[4] && preg_match('#^HTTP/\d+(?:\.\d+)? ([123456789]\d\d)(?: \|$)#', $h, $m)) {`
`257`	`257`	`if ($headers) {`
`258`	`258`	`$debug .= "< \r\n";`
`259`	`259`	`$headers = [];`