Merge pull request #60309 from nextcloud/backport/60262/stable33

[stable33] fix(http-client): detect brotli support via libcurl, not PHP extension

Author: ernolf

lib/private/Http/Client/Client.php

@@ -94,7 +94,7 @@ private function buildRequestOptions(array $options): array {
 		$headers = $options[RequestOptions::HEADERS] ?? [];
 		if (!isset($headers['Accept-Encoding'])) {
 			$acceptEnc = 'gzip';
-			if (function_exists('brotli_uncompress')) {
+			if (((curl_version()['features'] ?? 0) & CURL_VERSION_BROTLI) === CURL_VERSION_BROTLI) {
 				$acceptEnc = 'br, ' . $acceptEnc;
 			}
 			$options[RequestOptions::HEADERS] = $headers; // ensure headers are present

tests/lib/Http/Client/ClientTest.php

@@ -284,7 +284,7 @@ private function setUpDefaultRequestOptions(): void {
 		$this->serverVersion->method('getVersionString')
 			->willReturn('123.45.6');
 
-		$acceptEnc = function_exists('brotli_uncompress') ? 'br, gzip' : 'gzip';
+		$acceptEnc = (((curl_version()['features'] ?? 0) & CURL_VERSION_BROTLI) === CURL_VERSION_BROTLI) ? 'br, gzip' : 'gzip';
 		$this->defaultRequestOptions = [
 			'verify' => '/my/path.crt',
 			'proxy' => [
@@ -488,7 +488,7 @@ public function testSetDefaultOptionsWithNotInstalled(): void {
 		$this->serverVersion->method('getVersionString')
 			->willReturn('123.45.6');
 
-		$acceptEnc = function_exists('brotli_uncompress') ? 'br, gzip' : 'gzip';
+		$acceptEnc = (((curl_version()['features'] ?? 0) & CURL_VERSION_BROTLI) === CURL_VERSION_BROTLI) ? 'br, gzip' : 'gzip';
 
 		$this->assertEquals([
 			'verify' => \OC::$SERVERROOT . '/resources/config/ca-bundle.crt',
@@ -544,7 +544,7 @@ public function testSetDefaultOptionsWithProxy(): void {
 		$this->serverVersion->method('getVersionString')
 			->willReturn('123.45.6');
 
-		$acceptEnc = function_exists('brotli_uncompress') ? 'br, gzip' : 'gzip';
+		$acceptEnc = (((curl_version()['features'] ?? 0) & CURL_VERSION_BROTLI) === CURL_VERSION_BROTLI) ? 'br, gzip' : 'gzip';
 
 		$this->assertEquals([
 			'verify' => '/my/path.crt',
@@ -604,7 +604,7 @@ public function testSetDefaultOptionsWithProxyAndExclude(): void {
 		$this->serverVersion->method('getVersionString')
 			->willReturn('123.45.6');
 
-		$acceptEnc = function_exists('brotli_uncompress') ? 'br, gzip' : 'gzip';
+		$acceptEnc = (((curl_version()['features'] ?? 0) & CURL_VERSION_BROTLI) === CURL_VERSION_BROTLI) ? 'br, gzip' : 'gzip';
 
 		$this->assertEquals([
 			'verify' => '/my/path.crt',