App\Http\Middleware\LogDebugVisits

Complexity

Total Complexity

7

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	3
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	handle()	0	27	7

<?php

namespace App\Http\Middleware;

use Closure;
use Illuminate\Support\Arr;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Log;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Route;

class LogDebugVisits
{
    //  These items should never be logged
    protected $ignore = ['_token', 'token'];
    //  These items are logged with masked input to show that they did exist during the session
    protected $redact = ['password', 'password_confirmation'];


    /**
\     * When Debug Logging is one, log every page visit and all $request data
     */
    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);
    }
}