AbterPhp\Admin\Http\Middleware\CheckCsrfToken

Complexity

Total Complexity

6

Size/Duplication

Total Lines	84
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	isApiRequest()	0	9	1
A	__construct()	0	4	1
A	handle()	0	11	3
A	writeToResponse()	0	25	1

<?php

declare(strict_types=1);

namespace AbterPhp\Admin\Http\Middleware;

use AbterPhp\Admin\Config\Routes as RoutesConfig;
use Closure;
use Opulence\Framework\Configuration\Config;
use Opulence\Framework\Http\CsrfTokenChecker;
use Opulence\Http\InvalidCsrfTokenException;
use Opulence\Http\Requests\Request;
use Opulence\Http\Responses\Cookie;
use Opulence\Http\Responses\Response;
use Opulence\Routing\Middleware\IMiddleware;
use Opulence\Sessions\ISession;

/**
 * Defines the middleware that checks the CSRF token
 */
class CheckCsrfToken implements IMiddleware
{
    /** @var CsrfTokenChecker The CSRF token checker */
    protected $csrfTokenChecker = null;
    /** @var ISession The current session */
    protected $session = null;

    /**
     * @param CsrfTokenChecker $csrfTokenChecker The CSRF token checker
     * @param ISession         $session          The current session
     */
    public function __construct(CsrfTokenChecker $csrfTokenChecker, ISession $session)
    {
        $this->csrfTokenChecker = $csrfTokenChecker;
        $this->session          = $session;
    }

    /**
     * @inheritdoc
     * @throws InvalidCsrfTokenException Thrown if the CSRF token is invalid
     */
    public function handle(Request $request, Closure $next): Response
    {
        if ($this->isApiRequest($request)) {
            return $next($request);
        }

        if (!$this->csrfTokenChecker->tokenIsValid($request, $this->session)) {
            throw new InvalidCsrfTokenException('Invalid CSRF token');
        }

        return $this->writeToResponse($next($request));
    }

    /**
     * @param Request $request
     *
     * @return bool
     */
    protected function isApiRequest(Request $request): bool
    {
        $apiBasePath = RoutesConfig::getApiBasePath();

        $path = $request->getPath();

        $isApiRequest = substr($path, 0, strlen($apiBasePath)) === $apiBasePath;

        return $isApiRequest;
    }

    /**
     * @SuppressWarnings(PHPMD.StaticAccess)
     *
     * Writes data to the response
     *
     * @param Response $response The response to write to
     *
     * @return Response The response with the data written to it
     */
    protected function writeToResponse(Response $response): Response
    {
        $sessionValue = $this->session->get(CsrfTokenChecker::TOKEN_INPUT_NAME);

        $lifetime   = Config::get('sessions', 'xsrfcookie.lifetime');
        $expiration = time() + $lifetime;

        $path     = Config::get('sessions', 'cookie.path');
        $domain   = Config::get('sessions', 'cookie.domain');
        $isSecure = Config::get('sessions', 'cookie.isSecure');

        // Add an XSRF cookie for JavaScript frameworks to use
        $response->getHeaders()->setCookie(
            new Cookie(
                'XSRF-TOKEN',
                $sessionValue,
                $expiration,
                $path,

It seems like $path can also be of type null; however, parameter $path of Opulence\Http\Responses\Cookie::__construct() does only seem to accept string, maybe add an additional type check?

                $domain,

It seems like $domain can also be of type null; however, parameter $domain of Opulence\Http\Responses\Cookie::__construct() does only seem to accept string, maybe add an additional type check?

                $isSecure,

It seems like $isSecure can also be of type null; however, parameter $isSecure of Opulence\Http\Responses\Cookie::__construct() does only seem to accept boolean, maybe add an additional type check?

                false
            )
        );

        return $response;
    }
}