App\Http\Middleware\LogDebugVisits

Complexity

Total Complexity

8

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

86.67%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	handle()	0	32	8

<?php

namespace App\Http\Middleware;

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

class LogDebugVisits
{
    protected $ignore = ['_token'];
    protected $redact = ['password', 'password_confirmation'];

    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Closure  $next
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        if(config('logging.channels.daily.level') === 'debug')
        {
            $user = isset(Auth::user()->user_id) ? Auth::user()->full_name : \Request::ip();
            $requestData = $request->request->all();

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

            if($requestData)

The expression $requestData of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            {
                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);
        }

        return $next($request);
    }
}