Flipbox\OAuth2\Client\Provider\HubSpot

Complexity

Total Complexity

11

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	6

Test Coverage

Coverage

95.65%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	getBaseAuthorizationUrl()	0	4	1
A	getBaseAccessTokenUrl()	0	4	1
A	getResourceOwnerDetailsUrl()	0	4	1
A	getDefaultScopes()	0	4	1
A	getScopeSeparator()	0	4	1
A	getAuthorizationHeaders()	0	9	2
A	checkResponse()	0	8	3
A	createResourceOwner()	0	4	1

<?php

namespace Flipbox\OAuth2\Client\Provider;

use Flipbox\OAuth2\Client\Provider\Exception\HubSpotIdentityProviderException;
use League\OAuth2\Client\Provider\AbstractProvider;
use League\OAuth2\Client\Token\AccessToken;
use League\OAuth2\Client\Tool\BearerAuthorizationTrait;
use Psr\Http\Message\ResponseInterface;

class HubSpot extends AbstractProvider
{
    use BearerAuthorizationTrait;

    /**
     * Domain
     *
     * @var string
     */
    public $domain = 'https://app.hubspot.com';

    /**
     * Api domain
     *
     * @var string
     */
    public $apiDomain = 'https://api.hubapi.com';

    /**
     * @var array
     */
    protected $defaultScopes = [];

    /**
     * @inheritdoc
     */
    public function getBaseAuthorizationUrl()
    {
        return $this->domain . '/oauth/authorize';
    }

    /**
     * @inheritdoc
     */
    public function getBaseAccessTokenUrl(array $params)
    {
        return $this->apiDomain . '/oauth/v1/token';
    }

    /**
     * @inheritdoc
     */
    public function getResourceOwnerDetailsUrl(AccessToken $token)
    {
        return $this->apiDomain . '/oauth/v1/access-tokens/' . $token->getToken();
    }

    /**
     * @inheritdoc
     */
    protected function getDefaultScopes()
    {
        return $this->defaultScopes;
    }

    /**
     * @inheritdoc
     */
    protected function getScopeSeparator()
    {
        return ' ';
    }

    /**
     * @inheritdoc
     */
    protected function getAuthorizationHeaders($token = null)
    {
        if ($token === null) {
            return [];
        }
        return [
            'Authorization' => sprintf("Bearer %s", $token)
        ];
    }

    /**
     * @inheritdoc
     */
    protected function checkResponse(ResponseInterface $response, $data)
    {
        if ($response->getStatusCode() >= 400) {
            throw HubSpotIdentityProviderException::clientException($response, $data);

It seems like $data defined by parameter $data on line 90 can also be of type array; however, Flipbox\OAuth2\Client\Provider\Exception\HubSpotIdentityProviderException::clientException() does only seem to accept string, maybe add an additional type check?

        } elseif (isset($data['error'])) {
            throw HubSpotIdentityProviderException::oauthException($response, $data);

It seems like $data defined by parameter $data on line 90 can also be of type array; however, Flipbox\OAuth2\Client\Provider\Exception\HubSpotIdentityProviderException::oauthException() does only seem to accept string, maybe add an additional type check?

        }
    }

    /**
     * @inheritdoc
     */
    protected function createResourceOwner(array $response, AccessToken $token)
    {
        return new HubSpotResourceOwner($response);
    }
}