Audiens\AdobeClient\AuthAdobeTarget

Complexity

Total Complexity

6

Size/Duplication

Total Lines	103
Duplicated Lines	13.59 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	14	14	1
A	request()	0	14	1
A	send()	0	4	1
A	sendAsync()	0	4	1
A	requestAsync()	0	4	1
A	getConfig()	0	4	1

<?php

namespace Audiens\AdobeClient;

use Audiens\AdobeClient\Authentication\AuthStrategyInterface;
use Doctrine\Common\Cache\Cache;
use GuzzleHttp\Client;
use GuzzleHttp\ClientInterface;
use Psr\Http\Message\RequestInterface;

/**
 * Class Auth
 *
 */
class AuthAdobeTarget implements ClientInterface
{

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

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

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

    /** @var string */

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

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

    protected $authStrategy;

    /**
     * Auth constructor.
     * @param $clientId
     * @param $secretKey
     * @param $token
     * @param ClientInterface $clientInterface
     * @param AuthStrategyInterface $authStrategy
     */
    public function __construct(

This method seems to be duplicated in your project.

        $clientId,
        $secretKey,
        $token,
        ClientInterface $clientInterface,
        AuthStrategyInterface $authStrategy
    ) {
        $this->clientId = $clientId;
        $this->secretKey = $secretKey;
        $this->token = $token;

        $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;
    }


    /**
     *
     * This function is specific for adobe target request
     * @param       $method
     * @param null  $uri
     * @param array $options
     * @return mixed|\Psr\Http\Message\ResponseInterface
     * @throws \GuzzleHttp\Exception\GuzzleException
     */
    public function request($method, $uri = null, array $options = [])
    {
        $optionForToken = [
            'headers' => [
                'Authorization' =>  ['Bearer '.$this->authStrategy->authenticateJwtToken($this->clientId, $this->secretKey, $this->token, true)],
                'content-type' => 'application/vnd.adobe.target.v1+json',
                'x-api-key' => $this->clientId
            ],
        ];

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

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


    /**
     * @inheritDoc
     */
    public function send(RequestInterface $request, array $options = [])
    {
        return $this->client->send($request, $options);
    }

    /**
     * @inheritDoc
     */
    public function sendAsync(RequestInterface $request, array $options = [])
    {
        return $this->client->sendAsync($request, $options);
    }

    /**
     * @inheritDoc
     */
    public function requestAsync($method, $uri, array $options = [])
    {
        return $this->client->requestAsync($method, $uri, $options);

It seems like $uri defined by parameter $uri on line 105 can also be of type object<Psr\Http\Message\UriInterface>; however, GuzzleHttp\Client::requestAsync() does only seem to accept string, maybe add an additional type check?

    }

    /**
     * @inheritDoc
     */
    public function getConfig($option = null)
    {
        return $this->client->getConfig($option);
    }
}