Werk365\EtagConditionals\Middleware\IfMatch::handle()

Complexity

Conditions	4
Paths	3

Size

Total Lines	25
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	1	Features	0

<?php
    public function handle(Request $request, Closure $next)
    {
        // Next unless method is PATCH and If-Match header is set
        if (! ($request->isMethod('PATCH') && $request->hasHeader('If-Match'))) {
            return $next($request);
        }

        // Create new GET request to same endpoint,
        // copy headers and add header that allows you to ignore this request in middlewares
        $getRequest = Request::create($request->getRequestUri(), 'GET');
        $getRequest->headers = $request->headers;
        $getRequest->headers->set('X-Skip-Middleware', true);

true of type true is incompatible with the type string|string[] expected by parameter $values of Symfony\Component\HttpFoundation\HeaderBag::set().

        $getResponse = app()->handle($getRequest);

The method handle() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

        // Get content from response object and get hashes from content and etag
        $getContent = $getResponse->getContent();
        $getEtag = '"'.md5($getContent).'"';
        $ifMatch = $request->header('If-Match');

        // Compare current and request hashes
        if ($getEtag !== $ifMatch) {
            return response(null, 412);
        }

        return $next($request);
    }