CodeResponseType::handle() - Code Metrics - Inspection of "refactor" - php-guard/oauth-server - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 6e52f0...d9a404 )

by Alexandre

created 2018-03-04 18:01 UTC

CodeResponseType::handle() D

↳ Parent: CodeResponseType

Complexity

Conditions	10
Paths	18

Size

Total Lines	27
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	10
eloc	16
nc	18
nop	5
dl	0
loc	27
rs	4.8196
c	0
b	0
f	0

How to fix Complexity

<?php
/**
 * Created by PhpStorm.
 * User: GCC-MED
 * Date: 19/01/2018
 * Time: 15:36
 */

namespace OAuth2OLD\OpenID\ResponseTypes;


use OAuth2OLD\Config;
use OAuth2OLD\Exceptions\OAuthException;
use OAuth2OLD\OpenID\ResponseModes\ResponseModeInterface;
use OAuth2OLD\Repositories\ConfigurationRepository;
use OAuth2OLD\Roles\Clients\RegisteredClient;
use OAuth2OLD\Roles\ResourceOwnerInterface;
use OAuth2OLD\OpenID\Storages\AuthorizationCodeStorageInterface;
use Psr\Http\Message\ServerRequestInterface;

class CodeResponseType implements ResponseTypeInterface
{

    /**
     * @var ConfigurationRepository
     */
    private $configurationRepository;
    /**
     * @var AuthorizationCodeStorageInterface
     */
    private $authorizationCodeStorage;

    public function __construct(ConfigurationRepository $configurationRepository, AuthorizationCodeStorageInterface $authorizationCodeStorage)
    {
        $this->configurationRepository = $configurationRepository;
        $this->authorizationCodeStorage = $authorizationCodeStorage;
    }

    public function getResponseType(): string
    {
        return 'code';
    }

    /**
     * @param ServerRequestInterface $request
     * @param ResourceOwnerInterface $resourceOwner
     * @param RegisteredClient $client
     * @param array|null $scope
     * @param array|null $extendedResponseTypes
     * @return array
     * @throws OAuthException
     */
    public function handle(ServerRequestInterface $request, ResourceOwnerInterface $resourceOwner,
                           RegisteredClient $client, ?array $scope = null, ?array $extendedResponseTypes = null): array
    {
        $data = $request->getMethod() === 'GET' ? $request->getQueryParams() : $request->getParsedBody();

        if (is_array($client->getSupportedGrantTypes()) && !in_array('authorization_code', $client->getSupportedGrantTypes())) {
            throw new OAuthException('unauthorized_client',
                'Client is not authorized to request an authorization code with this method',
                'https://tools.ietf.org/html/rfc6749#section-5.2');
        }

        $idToken =

        $requestedScopes = $data['scope'] ?? null;
        $requestedScopes = $requestedScopes ? explode(' ', $requestedScopes) : [];

        $scopeRequestedIsIdentical = true;
        if ((empty($requestedScopes) && !is_null($scope)) || (is_array($scope) && !empty(array_diff($requestedScopes, $scope)))) {
            $scopeRequestedIsIdentical = false;
        }

        $scope = is_array($scope) ? implode(' ', $scope) : null;
        $authorizationCode = $this->authorizationCodeStorage->create(
            $client->getIdentifier(), $resourceOwner->getIdentifier(), $idToken, $data['redirect_uri'], $scope, $scopeRequestedIsIdentical);
            $client->getIdentifier(), $resourceOwner->getIdentifier(), /** @scrutinizer ignore-type */ $idToken, $data['redirect_uri'], $scope, $scopeRequestedIsIdentical);

        return [
            'code' => $authorizationCode->getCode()
        ];
    }


    /**
     * @param ServerRequestInterface $request
     */
    public function verifyRequest(ServerRequestInterface $request): void
    {
    }

    public function getDefaultResponseMode(): string
    {
        return ResponseModeInterface::RESPONSE_MODE_QUERY;
    }

    public function isImplicit(): bool
    {
        return false;
    }

    public function requireTLS(): bool
    {
        return true;
    }

    protected function generateCode()
    {
        return bin2hex(random_bytes(8));
    }

    public function verifyRequest(ServerRequestInterface $request): void
    {
    }

    public function isMultiValuedResponseTypeSupported(): bool
    {
        return true;
    }

    public function getExtendedResponseTypes(): ?array
    {
        return null;
    }

    public function isQueryResponseModeSupported(): bool
    {
        return true;
    }

}

php-guard / oauth-server

Push — master ( 6e52f0...d9a404 )

CodeResponseType::handle() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like