Dontdrinkandroot\RestBundle\Service\AccessTokenService

Complexity

Total Complexity

9

Size/Duplication

Total Lines	113
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
A	createAccessToken()	0	8	1
A	findUserByToken()	0	4	1
A	listByUser()	0	4	1
A	createAccessTokenForUser()	0	13	1
A	cleanUpExpiredTokens()	0	9	2
A	getDefaultExpirationDuration()	0	4	1
A	setDefaultExpirationDuration()	0	4	1

<?php

namespace Dontdrinkandroot\RestBundle\Service;

use Dontdrinkandroot\RestBundle\Entity\AccessToken;
use Dontdrinkandroot\RestBundle\Repository\AccessTokenRepositoryInterface;
use Symfony\Component\Security\Core\Authentication\AuthenticationManagerInterface;
use Symfony\Component\Security\Core\Authentication\Provider\AuthenticationProviderInterface;
use Symfony\Component\Security\Core\Authentication\Token\UsernamePasswordToken;
use Symfony\Component\Security\Core\User\UserInterface;

class AccessTokenService implements AccessTokenServiceInterface
{
    /**
     * @var AccessTokenRepositoryInterface
     */
    private $accessTokenRepository;

    /**
     * @var string
     */
    private $accessTokenClass;

    /**
     * @var string
     */
    private $defaultExpirationDuration = '+1 month';

    /**
     * @var AuthenticationProviderInterface
     */
    private $authenticationManager;

    /**
     * @var string
     */
    private $authenticationProviderKey;

    public function __construct(
        AccessTokenRepositoryInterface $accessTokenRepository,
        $accessTokenClass,
        AuthenticationManagerInterface $authenticationManager,
        $authenticationProviderKey
    ) {
        $this->accessTokenRepository = $accessTokenRepository;
        $this->accessTokenClass = $accessTokenClass;
        $this->authenticationManager = $authenticationManager;

$authenticationManager is of type object<Symfony\Component\Security\Core\Authentication\AuthenticationManagerInterface>, but the property $authenticationManager was declared to be of type object<Symfony\Component\Security\Core\Authentication\Provider\AuthenticationProviderInterface>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->authenticationProviderKey = $authenticationProviderKey;
    }

    /**
     * {@inheritdoc}
     */
    public function createAccessToken(string $username, string $password): AccessToken
    {
        $usernamePasswordToken = new UsernamePasswordToken($username, $password, $this->authenticationProviderKey);
        $token = $this->authenticationManager->authenticate($usernamePasswordToken);
        $accessToken = $this->createAccessTokenForUser($token->getUser());

        return $accessToken;
    }

    /**
     * {@inheritdoc}
     */
    public function findUserByToken(string $token): ?UserInterface
    {
        return $this->accessTokenRepository->findUserByToken($token);
    }

    /**
     * {@inheritdoc}
     */
    public function listByUser(UserInterface $user): array
    {
        return $this->accessTokenRepository->findBy(['user' => $user]);
    }

    /**
     * {@inheritdoc}
     */
    public function createAccessTokenForUser(UserInterface $user): AccessToken
    {
        $token = bin2hex(random_bytes(32));
        /** @var AccessToken $accessToken */
        $accessToken = new $this->accessTokenClass;
        $accessToken->setToken($token);
        $accessToken->setUser($user);
        $accessToken->setExpiry(new \DateTime($this->defaultExpirationDuration));

        $accessToken = $this->accessTokenRepository->persist($accessToken);

The method persist() does not seem to exist on object<Dontdrinkandroot\RestBundle\Repository\AccessTokenRepositoryInterface>.

        return $accessToken;
    }

    /**
     * {@inheritdoc}
     */
    public function cleanUpExpiredTokens(): int
    {
        $accessTokens = $this->accessTokenRepository->findExpiredTokens();
        foreach ($accessTokens as $accessToken) {
            $this->accessTokenRepository->remove($accessToken);

The method remove() does not seem to exist on object<Dontdrinkandroot\RestBundle\Repository\AccessTokenRepositoryInterface>.

        }

        return count($accessTokens);
    }

    /**
     * @return string
     */
    public function getDefaultExpirationDuration()
    {
        return $this->defaultExpirationDuration;
    }

    /**
     * @param string $defaultExpirationDuration
     */
    public function setDefaultExpirationDuration($defaultExpirationDuration)
    {
        $this->defaultExpirationDuration = $defaultExpirationDuration;
    }
}