[HttpClient] changes minimal php version to use curl push function

Anthony MARTIN · Anthony MARTIN · commit 01a663aea081 · 2019-03-18T16:11:55.000+01:00
diff --git a/src/Symfony/Component/HttpClient/CurlHttpClient.php b/src/Symfony/Component/HttpClient/CurlHttpClient.php
@@ -64,8 +64,8 @@ public function __construct(array $defaultOptions = [], int $maxHostConnections
             'dnsCache' => [[], [], []],
         ];
 
-        // Skip configuring HTTP/2 push when it's unsupported or buggy, see https://bugs.php.net/76675
-        if (\PHP_VERSION_ID < 70215 || \PHP_VERSION_ID === 70300 || \PHP_VERSION_ID === 70301) {
+        // Skip configuring HTTP/2 push when it's unsupported or buggy, see https://bugs.php.net/bug.php?id=77535
+        if (\PHP_VERSION_ID < 70217 || (\PHP_VERSION_ID >= 70300 && \PHP_VERSION_ID < 70304)) {
             return;
         }
 
@@ -74,8 +74,7 @@ public function __construct(array $defaultOptions = [], int $maxHostConnections
             return;
         }
 
-        // Keep a dummy "onPush" reference to work around a refcount bug in PHP
-        curl_multi_setopt($mh, CURLMOPT_PUSHFUNCTION, $multi->onPush = static function ($parent, $pushed, array $rawHeaders) use ($multi) {
+        curl_multi_setopt($mh, CURLMOPT_PUSHFUNCTION, static function ($parent, $pushed, array $rawHeaders) use ($multi) {
             return self::handlePush($parent, $pushed, $rawHeaders, $multi);
         });
     }

Original file line number	Diff line number	Diff line change
`@@ -64,8 +64,8 @@ public function __construct(array $defaultOptions = [], int $maxHostConnections`
`64`	`64`	`'dnsCache' => [[], [], []],`
`65`	`65`	`];`
`66`	`66`
`67`		`- // Skip configuring HTTP/2 push when it's unsupported or buggy, see https://bugs.php.net/76675`
`68`		`- if (\PHP_VERSION_ID < 70215 \|\| \PHP_VERSION_ID === 70300 \|\| \PHP_VERSION_ID === 70301) {`
	`67`	`+ // Skip configuring HTTP/2 push when it's unsupported or buggy, see https://bugs.php.net/bug.php?id=77535`
	`68`	`+ if (\PHP_VERSION_ID < 70217 \|\| (\PHP_VERSION_ID >= 70300 && \PHP_VERSION_ID < 70304)) {`
`69`	`69`	`return;`
`70`	`70`	`}`
`71`	`71`
`@@ -74,8 +74,7 @@ public function __construct(array $defaultOptions = [], int $maxHostConnections`
`74`	`74`	`return;`
`75`	`75`	`}`
`76`	`76`
`77`		`- // Keep a dummy "onPush" reference to work around a refcount bug in PHP`
`78`		`- curl_multi_setopt($mh, CURLMOPT_PUSHFUNCTION, $multi->onPush = static function ($parent, $pushed, array $rawHeaders) use ($multi) {`
	`77`	`+ curl_multi_setopt($mh, CURLMOPT_PUSHFUNCTION, static function ($parent, $pushed, array $rawHeaders) use ($multi) {`
`79`	`78`	`return self::handlePush($parent, $pushed, $rawHeaders, $multi);`
`80`	`79`	`});`
`81`	`80`	`}`