Custom Middleware - 'Too Many Redirects' - Laravel

Question

I want to create a custom middleware that only if the user is authenticated and the email is a certain email to access the /admin page.

Although, when I specify my custom route and then a redirect it always says too many redirects..

Short Explanation.

User Logs in -> redirected to /home. (Works)
If user tries to access /admin and their email isn't like the one specified in the middleware, redirect to /home.
If its true, let them in /admin

My middleware is called 'admin.verify'

Routes File:

Route::get('/admin', 'AdminController@index')->name('admin.index');

AdminController:

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;

class AdminController extends Controller
{
    public function __construct(){
      $this->middleware(['auth', 'admin.verify']);
    }


    public function index(){
      return view('admin.test');
    }
}

Middleware:

 public function handle($request, Closure $next)
    {

      if (Auth::check() && Auth::User()->email == '[email protected]') {
        return $next($request);
      } else {
        return redirect()->route('home');
      }

My Home Route:

 GET|HEAD | home | home| App\Http\Controllers\HomeController@index | web,auth

Home Controller:

class HomeController extends Controller
{
    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->middleware('auth');
    }

    /**
     * Show the application dashboard.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        return view('home');
    }
}

Your middleware should be within the Routes file not in the Controller — Option
@Option Why do you say that? Adding a middleware to a controller is valid. — Camilo
Where is the auth middleware redirecting to if the user is not authenticated? Could it be that is redirecting to the same home route? — Camilo

SUIFHUISHF SUIFHUISHF · Accepted Answer · 2017-11-01T16:07:09

Use $this->middleware ('admin.verify') instead of $this->middleware(['auth, admin.verify]');. You are getting too many redirects error because both the admin middleware and the constructor are checking if the user is authenticated.

Custom Middleware - 'Too Many Redirects' - Laravel

2 Answers