Merge pull request #159 from buckaroo-it/add-support-for-guzzlehttp-v5

BP-3290 Add support for guzzlehttp v5

src/Transaction/Client.php

@@ -31,7 +31,7 @@
 use Buckaroo\Services\TransactionHeaders\DefaultHeader;
 use Buckaroo\Services\TransactionHeaders\HmacHeader;
 use Buckaroo\Services\TransactionHeaders\SoftwareHeader;
-use Buckaroo\Transaction\Request\HttpClient\HttpClientGuzzle;
+use Buckaroo\Transaction\Request\HttpClient\HttpClientFactory;
 use Buckaroo\Transaction\Request\HttpClient\HttpClientInterface;
 use Buckaroo\Transaction\Request\Request;
 use Buckaroo\Transaction\Response\Response;
@@ -43,7 +43,7 @@ class Client
     private const METHOD_POST = 'POST';
 
     /**
-     * @var HttpClientInterface|HttpClientGuzzle
+     * @var HttpClientInterface
      */
     protected HttpClientInterface $httpClient;
     /**
@@ -61,7 +61,7 @@ class Client
     public function __construct(?Config $config)
     {
         $this->config = $config;
-        $this->httpClient = new HttpClientGuzzle($config->getLogger());
+        $this->httpClient = HttpClientFactory::createClient($config->getLogger());
     }
 
     /**

src/Transaction/Request/HttpClient/GuzzleHttpClientV5.php

@@ -0,0 +1,70 @@
+<?php
+
+namespace Buckaroo\Transaction\Request\HttpClient;
+
+use Buckaroo\Exceptions\BuckarooException;
+use Buckaroo\Exceptions\TransferException;
+use Buckaroo\Handlers\Logging\Subject;
+use GuzzleHttp\Client;
+use GuzzleHttp\ClientInterface;
+use GuzzleHttp\Exception\GuzzleException;
+use GuzzleHttp\Exception\RequestException;
+
+class GuzzleHttpClientV5 extends HttpClientAbstract
+{
+    /**
+     * @var Subject
+     */
+    protected Subject $logger;
+    protected ClientInterface $httpClient;
+
+    public function __construct(Subject $logger)
+    {
+        parent::__construct($logger);
+        $this->logger = $logger;
+
+        $this->httpClient = new Client([
+            'timeout' => self::TIMEOUT,
+            'connect_timeout' => self::CONNECT_TIMEOUT,
+        ]);
+    }
+
+    /**
+     * @param string $url
+     * @param array $headers
+     * @param string $method
+     * @param string|null $data
+     * @return array|mixed
+     * @throws TransferException
+     * @throws BuckarooException|GuzzleException
+     */
+
+    public function call(string $url, array $headers, string $method, string $data = null)
+    {
+        $headers = $this->convertHeadersFormat($headers);
+
+        $request = $this->httpClient->createRequest($method, $url, [
+            'headers' => $headers,
+            'body' => $data,
+        ]);
+
+        try
+        {
+            $response = $this->httpClient->send($request);
+
+            $result = (string) $response->getBody();
+
+            $this->logger->info('RESPONSE HEADERS: ' . json_encode($response->getHeaders()));
+            $this->logger->info('RESPONSE BODY: ' . $response->getBody());
+        } catch (RequestException $e) {
+            throw new TransferException($this->logger, "Transfer failed", 0, $e);
+        }
+
+        $result = $this->getDecodedResult($response, $result);
+
+        return [
+            $response,
+            $result,
+        ];
+    }
+}

src/Transaction/Request/HttpClient/HttpClientGuzzle.php → src/Transaction/Request/HttpClient/GuzzleHttpClientV7.php

@@ -1,27 +1,8 @@
 <?php
-/*
- * NOTICE OF LICENSE
- *
- * This source file is subject to the MIT License
- * It is available through the world-wide-web at this URL:
- * https://tldrlegal.com/license/mit-license
- * If you are unable to obtain it through the world-wide-web, please send an email
- * to support@buckaroo.nl so we can send you a copy immediately.
- *
- * DISCLAIMER
- *
- * Do not edit or add to this file if you wish to upgrade this module to newer
- * versions in the future. If you wish to customize this module for your
- * needs please contact support@buckaroo.nl for more information.
- *
- * @copyright Copyright (c) Buckaroo B.V.
- * @license   https://tldrlegal.com/license/mit-license
- */
-
-declare(strict_types=1);
 
 namespace Buckaroo\Transaction\Request\HttpClient;
 
+use Buckaroo\Exceptions\BuckarooException;
 use Buckaroo\Exceptions\TransferException;
 use Buckaroo\Handlers\Logging\Subject;
 use GuzzleHttp\Client;
@@ -30,7 +11,7 @@
 use GuzzleHttp\Psr7\Request;
 use GuzzleHttp\RequestOptions;
 
-class HttpClientGuzzle extends HttpClientAbstract
+class GuzzleHttpClientV7 extends HttpClientAbstract
 {
     /**
      * @var Subject
@@ -61,7 +42,7 @@ public function __construct(Subject $logger)
      * @param string|null $data
      * @return array|mixed
      * @throws TransferException
-     * @throws \Buckaroo\Exceptions\BuckarooException
+     * @throws BuckarooException
      */
     public function call(string $url, array $headers, string $method, string $data = null)
     {
@@ -77,8 +58,7 @@ public function call(string $url, array $headers, string $method, string $data =
 
             $this->logger->info('RESPONSE HEADERS: ' . json_encode($response->getHeaders()));
             $this->logger->info('RESPONSE BODY: ' . $response->getBody());
-        } catch (GuzzleException $e)
-        {
+        } catch (GuzzleException $e) {
             throw new TransferException($this->logger, "Transfer failed", 0, $e);
         }
 

src/Transaction/Request/HttpClient/HttpClientFactory.php

@@ -0,0 +1,23 @@
+<?php
+
+namespace Buckaroo\Transaction\Request\HttpClient;
+
+use Buckaroo\Handlers\Logging\Subject;
+use Composer\InstalledVersions;
+
+class HttpClientFactory
+{
+    public static function createClient(Subject $logger)
+    {
+        // Detect the installed GuzzleHttp version
+        $versionString  = InstalledVersions::getVersion('guzzlehttp/guzzle');
+        // Extract the major version number
+        $majorVersion = (int) explode('.', $versionString)[0];
+
+        // Instantiate the appropriate client based on the major version
+        if ($majorVersion === 5) {
+            return new GuzzleHttpClientV5($logger);
+        }
+        return new GuzzleHttpClientV7($logger);
+    }
+}