Bump minimum PHP version to 7.2 (#250)

* Package now requires PHP 7.2 or higher
* Changed dev dependency from Guzzle Adapter to straight Guzzle 7
* Drop PHP 7.1 from testing matrix
* Updated documentation

Author: dragonmantank

.github/workflows/build.yml

@@ -6,7 +6,7 @@ jobs:
             - ubuntu-latest
         strategy:
             matrix:
-                php: ['7.1', '7.2', '7.3', '7.4']
+                php: ['7.2', '7.3', '7.4']
         steps:
             - name: Configure Git
               if: ${{ matrix.os == 'windows-latest' }}

README.md

@@ -669,7 +669,7 @@ curl.cainfo = "C:\php\extras\ssl\cacert.pem"
 
 ### Pass custom HTTP client
 
-We allow use of any HTTPlug adapter, so you can create a client with alternative configuration if you need it, for example to take account of a local proxy, or deal with something else specific to your setup.
+We allow use of any HTTPlug adapter or PSR-18 compatible HTTP client, so you can create a client with alternative configuration if you need it, for example to take account of a local proxy, or deal with something else specific to your setup.
 
 Here's an example that reduces the default timeout to 5 seconds to avoid long delays if you have no route to our servers:
 
@@ -684,7 +684,7 @@ When things go wrong, you'll receive an `Exception`. The Vonage exception classe
 
 ### Composer installation fails due to Guzzle Adapter
 
-If you have a conflicting package installation that cannot co-exist with our recommended `php-http/guzzle6-adapter` package, then you may install the package `vonage/client-core` along with any package that satisfies the `php-http/client-implementation` requirement.
+If you have a conflicting package installation that cannot co-exist with our recommended `guzzlehttp/guzzle` package, then you may install the package `vonage/client-core` along with any package that satisfies the `php-http/client-implementation` requirement.
 
 See the [Packagist page for client-implementation](https://packagist.org/providers/php-http/client-implementation) for options.
 

composer.json

@@ -30,7 +30,7 @@
     "email": "[email protected]"
   },
   "require": {
-    "php": ">=7.1",
+    "php": ">=7.2",
     "psr/http-client-implementation": "^1.0",
     "laminas/laminas-diactoros": "^1.8.4 || ^2.0",
     "lcobucci/jwt": "^3.2",
@@ -43,7 +43,7 @@
     "php-http/mock-client": "^1.4",
     "estahn/phpunit-json-assertions": "^3.0.0",
     "squizlabs/php_codesniffer": "^3.1",
-    "php-http/guzzle6-adapter": "^2.0",
+    "guzzlehttp/guzzle": "^7.0",
     "phpstan/phpstan": "^0.12.11"
   },
   "autoload": {

src/Client.php

@@ -77,7 +77,13 @@ class Client
     public function __construct(CredentialsInterface $credentials, $options = array(), ClientInterface $client = null)
     {
         if (is_null($client)) {
-            $client = new \Http\Adapter\Guzzle6\Client();
+            // Since the user did not pass a client, try and make a client
+            // using the Guzzle 6 adapter or Guzzle 7
+            if (class_exists(\Http\Adapter\Guzzle6\Client::class)) {
+                $client = new \Http\Adapter\Guzzle6\Client();
+            } elseif (class_exists(\GuzzleHttp\Client::class)) {
+                $client = new \GuzzleHttp\Client();
+            }
         }
 
         $this->setHttpClient($client);

test/ClientTest.php

@@ -8,7 +8,7 @@
 
 namespace VonageTest;
 
-use Http\Adapter\Guzzle6\Client as HttpClient;
+use GuzzleHttp\Client as HttpClient;
 use Http\Message\MessageFactory\DiactorosMessageFactory;
 use Http\Mock\Client as HttpMock;
 use Vonage\Client;