nicoSWD\SecHeaderCheck\Domain\Header\URLScanner

Complexity

Total Complexity

6

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	parseHeader()	0	3	1
A	__construct()	0	8	1
A	processParsedHeaders()	0	3	1
A	getHeaders()	0	3	1
A	scanURL()	0	12	2

<?php declare(strict_types=1);

/**
 * @license  http://opensource.org/licenses/mit-license.php MIT
 * @link     https://github.com/nicoSWD
 * @author   Nicolas Oelgart <nico@oelgart.com>
 */
namespace nicoSWD\SecHeaderCheck\Domain\Header;

use nicoSWD\SecHeaderCheck\Domain\Result\AbstractParsedHeader;
use nicoSWD\SecHeaderCheck\Domain\Result\AuditionResult;
use nicoSWD\SecHeaderCheck\Domain\Result\ParsedHeaders;
use nicoSWD\SecHeaderCheck\Domain\Result\ScanResultProcessor;
use nicoSWD\SecHeaderCheck\Domain\URL\URL;
use nicoSWD\SecHeaderCheck\Domain\Validator\HeaderParserFactory;

final class URLScanner
{
    /** @var HeaderProviderInterface */
    private $headerProvider;
    /** @var HeaderParserFactory */
    private $parserFactory;
    /** @var ScanResultProcessor */
    private $scanResultProcessor;

    public function __construct(
        HeaderProviderInterface $headerProvider,
        HeaderParserFactory $parserFactory,
        ScanResultProcessor $scanResultProcessor
    ) {
        $this->headerProvider = $headerProvider;
        $this->parserFactory = $parserFactory;
        $this->scanResultProcessor = $scanResultProcessor;
    }

    public function scanURL(string $url, bool $followRedirects = true): AuditionResult
    {
        $headers = $this->getHeaders($url, $followRedirects);
        $parsedHeaders = new ParsedHeaders();

        foreach ($headers as $header) {
            $parsedHeaders->add(
                $this->parseHeader($header)

It seems like $header can also be of type boolean; however, parameter $header of nicoSWD\SecHeaderCheck\Domain\Header\URLScanner::parseHeader() does only seem to accept nicoSWD\SecHeaderCheck\Domain\Header\HttpHeader, maybe add an additional type check?

            );
        }

        return $this->processParsedHeaders($parsedHeaders);
    }

    private function getHeaders(string $url, bool $followRedirects): HttpHeaderBag
    {
        return $this->headerProvider->getHeadersFromUrl(new URL($url), $followRedirects);
    }

    private function parseHeader(HttpHeader $header): AbstractParsedHeader
    {
        return $this->parserFactory->createFromHeader($header)->parse();
    }

    private function processParsedHeaders(ParsedHeaders $scanResult): AuditionResult
    {
        return $this->scanResultProcessor->processParsedHeaders($scanResult);
    }
}