Ynlo\GraphQLBundle\Security\EndpointResolver

Complexity

Total Complexity

14

Size/Duplication

Total Lines	105
Duplicated Lines	0 %

Test Coverage

Coverage

96.77%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
C	resolveEndpoint()	0	45	12
A	cleanExpression()	0	3	1

<?php
/*******************************************************************************
 *  This file is part of the GraphQL Bundle package.
 *
 *  (c) YnloUltratech <support@ynloultratech.com>
 *
 *  For the full copyright and license information, please view the LICENSE
 *  file that was distributed with this source code.
 ******************************************************************************/

namespace Ynlo\GraphQLBundle\Security;

use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\Security\Core\Authorization\AuthorizationChecker;
use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
use Symfony\Component\Security\Core\Exception\AuthenticationCredentialsNotFoundException;
use Ynlo\GraphQLBundle\Definition\Registry\DefinitionRegistry;
use Ynlo\GraphQLBundle\Definition\Registry\Endpoint;
use Ynlo\GraphQLBundle\Extension\EndpointNotValidException;

class EndpointResolver
{
    /**
     * @var DefinitionRegistry
     */
    protected $definitionRegistry;

    /**
     * @var AuthorizationChecker
     */
    protected $authorizationChecker;

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

    /**
     * EndpointResolver constructor.
     *
     * Endpoints config should have the following format
     *
     * [
     * 'endpoints' => [
     *   'name' => [
     *      'roles'=> [],
     *      'host' => '',
     *      'path' => ''
     *    ]
     *  ]
     * ]
     *
     * @param DefinitionRegistry            $definitionRegistry
     * @param AuthorizationCheckerInterface $authorizationChecker
     * @param array                         $endpointsConfig
     */
    public function __construct(DefinitionRegistry $definitionRegistry, AuthorizationCheckerInterface $authorizationChecker, array $endpointsConfig = [])
    {
        $this->definitionRegistry = $definitionRegistry;
        $this->authorizationChecker = $authorizationChecker;

$authorizationChecker is of type Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface, but the property $authorizationChecker was declared to be of type Symfony\Component\Security\Core\Authorization\AuthorizationChecker. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->endpointsConfig = $endpointsConfig['endpoints'] ?? [];
    }

    /**
     * @param Request $request
     *
     * @return null|Endpoint
     *
     * @throws EndpointNotValidException
     */
    public function resolveEndpoint(Request $request): ?Endpoint
    {
        if (empty($this->endpointsConfig)) {
            return $this->definitionRegistry->getEndpoint();
        }

        foreach ($this->endpointsConfig as $endpoint => $config) {
            if (isset($config['host'])) {
                $host = $request->getHost();
                if (preg_match(sprintf('/%s/', $this->cleanExpression($config['host'])), $host)) {

It seems like $this->cleanExpression($config['host']) can also be of type string[]; however, parameter $args of sprintf() does only seem to accept string, maybe add an additional type check?

                    $hostPassed = true;
                } else {
                    $hostPassed = false;
                }
            } else {
                $hostPassed = true;
            }

            if (isset($config['path'])) {
                $path = $request->getPathInfo();
                if (preg_match(sprintf('/%s/', $this->cleanExpression($config['path'])), $path)) {
                    $pathPassed = true;
                } else {
                    $pathPassed = false;
                }
            } else {
                $pathPassed = true;
            }

            if (isset($config['roles'])) {
                try {
                    $rolePassed = $this->authorizationChecker->isGranted($config['roles']);
                } catch (AuthenticationCredentialsNotFoundException $exception) {
                    $rolePassed = false;
                }
            } else {
                $rolePassed = true;
            }

            if ($rolePassed && $hostPassed && $pathPassed) {
                return $this->definitionRegistry->getEndpoint($endpoint);
            }
        }

        return null;
    }

    /**
     * @param string $exp
     *
     * @return null|string|string[]
     */
    private function cleanExpression($exp)
    {
        return preg_replace('/\//', '\/', $exp);
    }
}