App\Http\Middleware\LogDebugVisits::handle()

Complexity

Conditions	7
Paths	4

Size

Total Lines	27
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	7

Importance

Changes	3
Bugs	0	Features	0

<?php
    public function handle(Request $request, Closure $next)
    {
        $user = isset(Auth::user()->user_id) ? Auth::user()->full_name : \Request::ip();

Accessing user_id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

Accessing full_name on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        $requestData = $request->request->all();

        Log::debug('Route '.Route::currentRouteName().' visited by '.$user);

        if(!empty($requestData))
        {
            foreach($this->redact as $i)
            {
                if(Arr::exists($requestData, $i))
                {
                    $requestData[$i] = '[REDACTED]';
                }
            }
            foreach($this->ignore as $i)
            {
                if(Arr::exists($requestData, $i))
                {
                    unset($requestData[$i]);
                }
            }
            Log::debug('Submitted Data ', $requestData);
        }

        return $next($request);
    }