App\Http\Middleware\Authenticate

Complexity

Total Complexity

9

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	redirectTo()	0	9	3
A	authenticate()	0	14	6

<?php

namespace App\Http\Middleware;

use Illuminate\Auth\Middleware\Authenticate as Middleware;

class Authenticate extends Middleware
{
    /**
     * Get the path the user should be redirected to when they are not authenticated.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return string|null
     */
    protected function redirectTo($request)
    {
        if($request->has('sso')){
            session()->put('sso', $request->get('sso'));
            session()->put('sig', $request->get('sig'));
        }

        if (! $request->expectsJson()) {
            return route('login');
        }
    }

    protected function authenticate($request, array $guards)
    {
        if (empty($guards)) {
            $guards = [null];
        }
        foreach ($guards as $guard) {
            if ($this->auth->guard($guard)->check()) {
                return $this->auth->shouldUse($guard);

Are you sure the usage of $this->auth->shouldUse($guard) targeting Illuminate\Auth\AuthManager::shouldUse() seems to always return null.

            }
            if ($request->input('wiki_session_id') && $request->input('wiki_session_id') !== null) {
                return;
            }
        }
        $this->unauthenticated($request, $guards);
    }
}