Surfnet\StepupSelfService\SelfServiceBundle\Security\Authentication\EventSubscriber\CustomLogoutListener

Complexity

Total Complexity

3

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	onLogout()	0	6	1
A	getSubscribedEvents()	0	4	1

<?php

Missing file doc comment

declare(strict_types = 1);

namespace Surfnet\StepupSelfService\SelfServiceBundle\Security\Authentication\EventSubscriber;

use Symfony\Component\HttpFoundation\RedirectResponse;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
use Symfony\Component\Security\Http\Event\LogoutEvent;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;

readonly class CustomLogoutListener implements EventSubscriberInterface

Missing doc comment for class CustomLogoutListener

{
    public static function getSubscribedEvents(): array

Missing doc comment for function getSubscribedEvents()

    {
        return [
            LogoutEvent::class => 'onLogout',
        ];
    }

    public function __construct(

Missing doc comment for function __construct()

        private TokenStorageInterface $tokenStorage,
        private array $logoutRedirectUrl,
    ) {
    }
    public function onLogout(): RedirectResponse

Missing doc comment for function onLogout()

    {
        $token    = $this->tokenStorage->getToken();
        $identity = $token->getUser();

        return new RedirectResponse($this->logoutRedirectUrl[$identity->preferredLocale]);

Accessing preferredLocale on the interface Symfony\Component\Security\Core\User\UserInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

    }
}