Do not extend Guzzle\Client

Extending Guzzle\Client is deprecated since Guzzle 7.1 and the class is annotated as @Final. The class will be actually final in Guzzle 8.0

Author: ThibaultVlacich

README.md

@@ -40,7 +40,7 @@ $twitch_client_id = 'TWITCH_CLIENT_ID';
 $twitch_client_secret = 'TWITCH_CLIENT_SECRET';
 $twitch_scopes = '';
 
-$helixGuzzleClient = new \NewTwitchApi\HelixGuzzleClient($twitch_client_id);
+$helixGuzzleClient = \NewTwitchApi\HelixGuzzleClient::getClient($twitch_client_id);
 $newTwitchApi = new \NewTwitchApi\NewTwitchApi($helixGuzzleClient, $twitch_client_id, $twitch_client_secret);
 $oauth = $newTwitchApi->getOauthApi();
 
@@ -62,7 +62,7 @@ $twitch_client_id = 'TWITCH_CLIENT_ID';
 $twitch_client_secret = 'TWITCH_CLIENT_SECRET';
 $twitch_scopes = '';
 
-$helixGuzzleClient = new \NewTwitchApi\HelixGuzzleClient($twitch_client_id);
+$helixGuzzleClient = \NewTwitchApi\HelixGuzzleClient::getClient($twitch_client_id);
 $newTwitchApi = new \NewTwitchApi\NewTwitchApi($helixGuzzleClient, $twitch_client_id, $twitch_client_secret);
 $oauth = $newTwitchApi->getOauthApi();
 
@@ -104,7 +104,7 @@ $twitch_client_secret = 'TWITCH_CLIENT_SECRET';
 $twitch_scopes = '';
 $user_refresh_token = 'REFRESH_TOKEN';
 
-$helixGuzzleClient = new \NewTwitchApi\HelixGuzzleClient($twitch_client_id);
+$helixGuzzleClient = \NewTwitchApi\HelixGuzzleClient::getClient($twitch_client_id);
 $newTwitchApi = new \NewTwitchApi\NewTwitchApi($helixGuzzleClient, $twitch_client_id, $twitch_client_secret);
 $oauth = $newTwitchApi->getOauthApi();
 
@@ -140,7 +140,7 @@ $twitch_access_token = 'the token';
 
 // The Guzzle client used can be the included `HelixGuzzleClient` class, for convenience.
 // You can also use a mock, fake, or other double for testing, of course.
-$helixGuzzleClient = new HelixGuzzleClient($twitch_client_id);
+$helixGuzzleClient = \NewTwitchApi\HelixGuzzleClient::getClient($twitch_client_id);
 
 // Instantiate NewTwitchApi. Can be done in a service layer and injected as well.
 $newTwitchApi = new NewTwitchApi($helixGuzzleClient, $twitch_client_id, $twitch_client_secret);

src/NewTwitchApi/Auth/AuthGuzzleClient.php

@@ -6,13 +6,13 @@
 
 use GuzzleHttp\Client;
 
-class AuthGuzzleClient extends Client
+class AuthGuzzleClient
 {
     private const BASE_URI = 'https://id.twitch.tv/oauth2/';
 
-    public function __construct(array $config = [])
+    public static function getClient(array $config = []): Client
     {
-        parent::__construct($config + [
+        return new Client($config + [
             'base_uri' => self::BASE_URI,
         ]);
     }

src/NewTwitchApi/Auth/OauthApi.php

@@ -20,7 +20,7 @@ public function __construct(string $clientId, string $clientSecret, Client $guzz
     {
         $this->clientId = $clientId;
         $this->clientSecret = $clientSecret;
-        $this->guzzleClient = $guzzleClient ?? new AuthGuzzleClient();
+        $this->guzzleClient = $guzzleClient ?? AuthGuzzleClient::getClient();
     }
 
     /**

src/NewTwitchApi/HelixGuzzleClient.php

@@ -6,13 +6,13 @@
 
 use GuzzleHttp\Client;
 
-class HelixGuzzleClient extends Client
+class HelixGuzzleClient
 {
     private const BASE_URI = 'https://api.twitch.tv/helix/';
 
-    public function __construct(string $clientId, array $config = [])
+    public static function getClient(string $clientId, array $config = []): Client
     {
-        parent::__construct($config + [
+        return new Client($config + [
             'base_uri' => self::BASE_URI,
             'timeout' => 30,
             'headers' => ['Client-ID' => $clientId, 'Content-Type' => 'application/json'],

src/NewTwitchApi/Webhooks/WebhooksSubscriptionApi.php

@@ -20,7 +20,7 @@ public function __construct(string $clientId, string $secret, Client $guzzleClie
     {
         $this->clientId = $clientId;
         $this->secret = $secret;
-        $this->guzzleClient = $guzzleClient ?? new HelixGuzzleClient($clientId);
+        $this->guzzleClient = $guzzleClient ?? HelixGuzzleClient::getClient($clientId);
     }
 
     public function subscribeToStream(string $twitchId, string $callback, string $bearer, int $leaseSeconds = 0): void