Spatie\Csp\AddCspHeaders

Complexity

Total Complexity

5

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	11	1
B	getProfiles()	0	30	4

<?php

namespace Spatie\Csp;

use Closure;
use Illuminate\Http\Request;
use Illuminate\Support\Collection;

class AddCspHeaders
{
    public function handle(Request $request, Closure $next, $customProfileClass = null)
    {
        $response = $next($request);

        $this
            ->getProfiles($customProfileClass, $response)
            ->filter->shouldBeApplied($request, $response)
            ->each->applyTo($response);

        return $response;
    }

    protected function getProfiles(string $customProfileClass = null, $response): Collection

The parameter $response is not used and could be removed.

    {
        $profiles = collect();

        if ($customProfileClass) {

The expression $customProfileClass of type null|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

            $profiles->push(ProfileFactory::create($customProfileClass));

            return $profiles;
        }

        $profileClass = config('csp.profile');

        if (!empty($profileClass)) {

            $profiles->push(ProfileFactory::create($profileClass));

        }

        $reportOnlyProfileClass = config('csp.report_only_profile');

        if (!empty($reportOnlyProfileClass)) {
            $profile = ProfileFactory::create($reportOnlyProfileClass);

            $profile->reportOnly();

            $profiles->push($profile);
        }

        return $profiles;
    }
}