LoginCidadao\RemoteClaimsBundle\EventSubscriber\AuthorizationSubscriber

Complexity

Total Complexity

12

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getSubscribedEvents()	0	6	1
B	onNewAuthorizationRequest()	0	17	5
A	checkHttpUri()	0	10	3
A	checkTagUri()	0	10	2

<?php
/**
 * This file is part of the login-cidadao project or it's bundles.
 *
 * (c) Guilherme Donato <guilhermednt on github>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace LoginCidadao\RemoteClaimsBundle\EventSubscriber;

use LoginCidadao\LogBundle\Traits\LoggerAwareTrait;
use LoginCidadao\OpenIDBundle\Event\AuthorizationEvent;
use LoginCidadao\OpenIDBundle\LoginCidadaoOpenIDEvents;
use LoginCidadao\RemoteClaimsBundle\Model\HttpUri;
use LoginCidadao\RemoteClaimsBundle\Model\RemoteClaimFetcherInterface;
use LoginCidadao\RemoteClaimsBundle\Model\RemoteClaimInterface;
use LoginCidadao\RemoteClaimsBundle\Model\TagUri;
use Psr\Log\LoggerAwareInterface;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;

class AuthorizationSubscriber implements EventSubscriberInterface, LoggerAwareInterface
{
    use LoggerAwareTrait;

    /** @var RemoteClaimFetcherInterface */
    private $claimFetcher;

    /** @var RemoteClaimInterface[] */
    private $remoteClaims;

    /**
     * AuthorizationSubscriber constructor.
     * @param RemoteClaimFetcherInterface $claimFetcher
     */
    public function __construct(RemoteClaimFetcherInterface $claimFetcher)
    {
        $this->claimFetcher = $claimFetcher;
    }

    public static function getSubscribedEvents()
    {
        return [
            LoginCidadaoOpenIDEvents::NEW_AUTHORIZATION_REQUEST => 'onNewAuthorizationRequest',
        ];
    }

    public function onNewAuthorizationRequest(AuthorizationEvent $event)
    {
        foreach ($event->getScope() as $scope) {
            if ($this->checkHttpUri($scope) || $this->checkTagUri($scope)) {
                try {
                    $remoteClaim = $this->claimFetcher->getRemoteClaim($scope);
                    $this->remoteClaims[] = $remoteClaim;
                    $event->addRemoteClaim($remoteClaim);
                } catch (\Exception $e) {
                    $this->error(
                        "Error retrieving remote claim {$scope}: {$e->getMessage()}",
                        ['exception' => $e]
                    );
                }
            }
        }
    }

    private function checkHttpUri($uri)

function checkHttpUri() does not seem to conform to the naming convention (^(?:is|has|should|may|supports)).

    {
        try {
            $http = HttpUri::createFromString($uri);

            return $http->getScheme() && $http->getHost();
        } catch (\Exception $e) {
            return false;
        }
    }

    private function checkTagUri($uri)

function checkTagUri() does not seem to conform to the naming convention (^(?:is|has|should|may|supports)).

    {
        try {
            TagUri::createFromString($uri);

            return true;
        } catch (\Exception $e) {
            return false;
        }
    }
}