Audiens\DoubleclickClient\Auth

Complexity

Total Complexity

2

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	request()	0	13	1

<?php

namespace Audiens\DoubleclickClient;

use Audiens\DoubleclickClient\authentication\AuthStrategyInterface;
use Doctrine\Common\Cache\Cache;
use GuzzleHttp\Client;
use GuzzleHttp\ClientInterface;

/**
 * Class Auth
 */
class Auth extends Client implements ClientInterface
{

    /**
 * @var  Cache
*/
    protected $cache;

    /**
 * @var  Client
*/
    protected $client;

    /**
 * @var string
*/
    protected $token;

    /**
 * @var AuthStrategyInterface
*/
    protected $authStrategy;

    /**
     * Auth constructor.
     *
     * @param ClientInterface       $clientInterface
     * @param AuthStrategyInterface $authStrategy
     */
    public function __construct(ClientInterface $clientInterface, AuthStrategyInterface $authStrategy)
    {
        $this->client = $clientInterface;

$clientInterface is of type object<GuzzleHttp\ClientInterface>, but the property $client was declared to be of type object<GuzzleHttp\Client>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->authStrategy = $authStrategy;

        parent::__construct([]);
    }

    /**
     * @param string $method
     * @param null   $uri
     * @param array  $options
     *
     * @return mixed|\Psr\Http\Message\ResponseInterface
     * @throws \Exception
     */
    public function request($method, $uri = null, array $options = [])
    {

        $optionForToken = [
            'headers' => [
                'Authorization' => (string)$this->authStrategy->authenticate(),
            ],
        ];

        $options = array_merge($options, $optionForToken);

        return $this->client->request($method, $uri, $options);
    }
}