Merge pull request #33087 from nextcloud/fix/30282/external-storages

fix external storages access

lib/private/Http/Client/ClientService.php

@@ -28,7 +28,6 @@ namespace OC\Http\Client;
 
 use GuzzleHttp\Client as GuzzleClient;
 use GuzzleHttp\HandlerStack;
-use GuzzleHttp\Handler\CurlHandler;
 use OCP\Http\Client\IClient;
 use OCP\Http\Client\IClientService;
 use OCP\ICertificateManager;
@@ -63,8 +62,7 @@ class ClientService implements IClientService {
 	 * @return Client
 	 */
 	public function newClient(): IClient {
-		$handler = new CurlHandler();
+		$stack = HandlerStack::create();
-		$stack = HandlerStack::create($handler);
 		$stack->push($this->dnsPinMiddleware->addDnsPinning());
 
 		$client = new GuzzleClient(['handler' => $stack]);

tests/lib/Http/Client/ClientServiceTest.php

@@ -10,7 +10,6 @@ namespace Test\Http\Client;
 
 use GuzzleHttp\Client as GuzzleClient;
 use GuzzleHttp\HandlerStack;
-use GuzzleHttp\Handler\CurlHandler;
 use OC\Http\Client\Client;
 use OC\Http\Client\ClientService;
 use OC\Http\Client\DnsPinMiddleware;
@@ -42,8 +41,7 @@ class ClientServiceTest extends \Test\TestCase {
 			$localAddressChecker
 		);
 
-		$handler = new CurlHandler();
+		$stack = HandlerStack::create();
-		$stack = HandlerStack::create($handler);
 		$stack->push($dnsPinMiddleware->addDnsPinning());
 		$guzzleClient = new GuzzleClient(['handler' => $stack]);