From ba32a665f17a7c9f973c726c041a85b457a18709 Mon Sep 17 00:00:00 2001 From: James Cole Date: Fri, 8 Jan 2016 16:00:57 +0100 Subject: [PATCH] Update middleware after upgrade to Laravel 5.2 --- app/Http/Middleware/Authenticate.php | 52 ++++------------- app/Http/Middleware/EncryptCookies.php | 17 ++++++ app/Http/Middleware/Range.php | 2 +- .../Middleware/RedirectIfAuthenticated.php | 43 ++++---------- app/Http/Middleware/ReplaceTestVars.php | 57 ------------------- app/Http/Middleware/VerifyCsrfToken.php | 28 +++------ 6 files changed, 47 insertions(+), 152 deletions(-) mode change 100644 => 100755 app/Http/Middleware/Authenticate.php create mode 100755 app/Http/Middleware/EncryptCookies.php mode change 100644 => 100755 app/Http/Middleware/RedirectIfAuthenticated.php delete mode 100644 app/Http/Middleware/ReplaceTestVars.php diff --git a/app/Http/Middleware/Authenticate.php b/app/Http/Middleware/Authenticate.php old mode 100644 new mode 100755 index 5cc1d665de..e7424954b4 --- a/app/Http/Middleware/Authenticate.php +++ b/app/Http/Middleware/Authenticate.php @@ -1,65 +1,33 @@ -auth = $auth; - } - /** * Handle an incoming request. * * @param \Illuminate\Http\Request $request * @param \Closure $next + * @param string|null $guard * * @return mixed */ - public function handle(Request $request, Closure $next) + public function handle($request, Closure $next, $guard = null) { - - if ($this->auth->guest()) { + if (Auth::guard($guard)->guest()) { if ($request->ajax()) { return response('Unauthorized.', 401); } else { - return redirect()->guest('auth/login'); - } - } - /** @var User $user */ - $user = $this->auth->user(); - if ($user instanceof User && intval($user->blocked) == 1) { - Auth::logout(); - return redirect()->route('index'); + return redirect()->guest('login'); + } } // if logged in, set user language: @@ -72,7 +40,7 @@ class Authenticate setlocale(LC_TIME, $locale); setlocale(LC_MONETARY, $locale); + return $next($request); } - } diff --git a/app/Http/Middleware/EncryptCookies.php b/app/Http/Middleware/EncryptCookies.php new file mode 100755 index 0000000000..d863b91383 --- /dev/null +++ b/app/Http/Middleware/EncryptCookies.php @@ -0,0 +1,17 @@ +auth->check()) { diff --git a/app/Http/Middleware/RedirectIfAuthenticated.php b/app/Http/Middleware/RedirectIfAuthenticated.php old mode 100644 new mode 100755 index b5419be62c..5fbd2d05b2 --- a/app/Http/Middleware/RedirectIfAuthenticated.php +++ b/app/Http/Middleware/RedirectIfAuthenticated.php @@ -1,52 +1,29 @@ -auth = $auth; - } - /** * Handle an incoming request. * * @param \Illuminate\Http\Request $request * @param \Closure $next + * @param string|null $guard * * @return mixed */ - public function handle(Request $request, Closure $next) + public function handle($request, Closure $next, $guard = null) { - if ($this->auth->check()) { - return new RedirectResponse(url('/')); + if (Auth::guard($guard)->check()) { + + return redirect('/'); } return $next($request); } - } diff --git a/app/Http/Middleware/ReplaceTestVars.php b/app/Http/Middleware/ReplaceTestVars.php deleted file mode 100644 index 5e90f0253c..0000000000 --- a/app/Http/Middleware/ReplaceTestVars.php +++ /dev/null @@ -1,57 +0,0 @@ -app = $app; - } - - /** - * Handle an incoming request. - * - * @param \Illuminate\Http\Request $request - * @param \Closure $next - * - * @return mixed - */ - public function handle(Request $request, Closure $next) - { - if ('testing' === $this->app->environment() && $request->has('_token')) { - $input = $request->all(); - $input['_token'] = $request->session()->token(); - // we need to update _token value to make sure we get the POST / PUT tests passed. - Log::debug('Input token replaced (' . $input['_token'] . ').'); - $request->replace($input); - } - - return $next($request); - } - -} diff --git a/app/Http/Middleware/VerifyCsrfToken.php b/app/Http/Middleware/VerifyCsrfToken.php index 71dd254d3d..89a57ce341 100644 --- a/app/Http/Middleware/VerifyCsrfToken.php +++ b/app/Http/Middleware/VerifyCsrfToken.php @@ -1,27 +1,17 @@ -