App\EventSubscriber\SecuritySubscriber

Complexity

Total Complexity

5

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	onAuthenticationSuccess()	0	5	2
A	onAuthenticationFailure()	0	6	1
A	getSubscribedEvents()	0	5	1
A	__construct()	0	3	1

<?php

namespace App\EventSubscriber;

use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\Security\Core\AuthenticationEvents;
use Symfony\Component\Security\Core\Event\AuthenticationFailureEvent;
use Symfony\Component\Security\Core\Event\AuthenticationEvent;
use Psr\Log\LoggerInterface;


/**
 * This event subscriber monitors and logs user authentication attempts.
 *
 */
class SecuritySubscriber implements EventSubscriberInterface
{
    public static function getSubscribedEvents()
    {
        return array(
            AuthenticationEvents::AUTHENTICATION_FAILURE => 'onAuthenticationFailure',
            AuthenticationEvents::AUTHENTICATION_SUCCESS => 'onAuthenticationSuccess',
        );
    }

    public function __construct(LoggerInterface $logger)
    {
        $this->logger = $logger;

The property logger does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    public function onAuthenticationFailure(AuthenticationFailureEvent $event)
    {
        $exception = $event->getAuthenticationException();
        $token = $event->getAuthenticationToken();
        $creds = $token->getCredentials();
        $this->logger->error("Login failed for " . $creds['username'] . ": " . $exception->getMessage());
    }

    public function onAuthenticationSuccess(AuthenticationEvent $event)
    {
        $token = $event->getAuthenticationToken();
        if ($token->getUsername() != "anon.") {
            $this->logger->info("Login succeeded: " . $token->getUsername());
        }
    }
}