App\Http\Middleware\ForceJsonOnAPI

Complexity

Total Complexity

3

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	8	3

<?php

namespace App\Http\Middleware;

use Closure;
use Illuminate\Http\Request;
use Illuminate\Support\Str;
use Symfony\Component\HttpFoundation\Response;

class ForceJsonOnAPI
{
    /**
     * Handle an incoming request.
     *
     * @param  \Closure(\Illuminate\Http\Request): (\Symfony\Component\HttpFoundation\Response)  $next
     */
    public function handle(Request $request, Closure $next): Response
    {
        // Force Json accept type on api routes
        if ($request->is('api/*') && ! Str::contains($request->header('accept'), ['/json', '+json'])) {

It seems like $request->header('accept') can also be of type array; however, parameter $haystack of Illuminate\Support\Str::contains() does only seem to accept string, maybe add an additional type check?

            $request->headers->set('accept', 'application/json,'.$request->header('accept'));

Are you sure $request->header('accept') of type array|null|string can be used in concatenation?

        }

        return $next($request);
    }
}