example\components\HttpBasicAuthMiddleware

Complexity

Total Complexity

10

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	parseHeader()	0	9	3
A	process()	0	12	2
A	login()	0	16	4

<?php

namespace example\components;

use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;
use Psr\Http\Server\MiddlewareInterface;
use Psr\Http\Server\RequestHandlerInterface;
use Zend\Diactoros\Response;

class HttpBasicAuthMiddleware implements MiddlewareInterface
{
    /**
     * @var array
     */
    private $users;

    /**
     * HttpBasicAuthMiddleware constructor.
     * @param array $users
     */
    public function __construct(array $users)
    {
        $this->users = $users;
    }

    /**
     * @param ServerRequestInterface $request
     * @param RequestHandlerInterface $handler
     * @return ResponseInterface
     */
    public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
    {
        $login = $this->login($request);

        if (empty($login)) {
            $response = new Response('php://memory');
            return $response->withStatus(401, 'Unauthorized')->withHeader('WWW-Authenticate', 'Basic realm="Test"');
        }

        $response = $handler->handle($request);

        return $response;
    }

    /**
     * Check the user credentials and return the username or false.
     *
     * @param ServerRequestInterface $request
     * @return bool|mixed
     */
    private function login(ServerRequestInterface $request)
    {
        //Check header
        $authorization = $this->parseHeader($request->getHeaderLine('Authorization'));

        if (!$authorization) {

$authorization is a non-empty array, thus ! $authorization is always false.

The expression $authorization of type array<string,null|string> is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return false;
        }
        //Check the user
        if (!isset($this->users[$authorization['username']])) {
            return false;
        }
        if ($this->users[$authorization['username']] !== $authorization['password']) {
            return false;
        }
        return $authorization['username'];
    }

    /**
     * Parses the authorization header for a basic authentication.
     *
     * @param string $header
     * @return array|bool
     */
    private function parseHeader(string $header)
    {
        if (strpos($header, 'Basic') !== 0) {
            return false;
        }
        $header = explode(':', base64_decode(substr($header, 6)), 2);
        return [
            'username' => $header[0],
            'password' => isset($header[1]) ? $header[1] : null,
        ];
    }
}