This page isn’t working localhost redirected you too many times. Try clearing your cookies. ERR_TOO_MANY_REDIRECTS

Question

hello I've been struggling with this error since yesterday, I need some help here is my middleware and here my route. This page isn’t working localhost redirected you too many times. Try clearing your cookies. ERR_TOO_MANY_REDIRECTS.

    public function handle($request, Closure $next)
    {
        if (Auth::check()) {
            $user = Auth::user();
            if ($user->hasAnyRole('school')) {
                return $next($request);
            }
        } else {
            return redirect('login'); 
        }
    }

Route::get('/', 'PagesController@index');
Route::get('/about', 'PagesController@about')->name('about');

Auth::routes();
Route::get('/school', 'HomeController@index')->name('school')->middleware('school');

It's even impossible to access the landing page. Thanks in advance!

just to detect the problem can you temporary remove ``` else { return redirect('login'); } ``` in middleware check if still have this issue or not — TEFO, Jun 21 '20 at 16:30
I removed it and I got this instead Error Call to a member function send() on null — Ibrahim, Jun 21 '20 at 16:35
You have to check which block causes your redirect, use `dd()` for this — Dev Man, Jun 21 '20 at 17:03
I mean does the route `Route::get('/', 'PagesController@index');` work? — apokryfos, Jun 21 '20 at 19:12

A.A Noman · Answer 1 · 2020-06-21T17:05:35.827

0

You have to use like this

public function __construct(){
    public function handle($request, Closure $next)
        if (Auth::check()) {
            $user = Auth::user();
            if ($user->hasAnyRole('school')) {
                return $next($request);
            }
            else{
                // Another redirect
            }
        } else {
            return redirect()->route('login');
        }
    });
}

edited Jun 21 '20 at 17:05

answered Jun 21 '20 at 16:46

A.A Noman

5,244
9
24
46

I did use it and it didn't work either. In fact wherever I redirect I have the same error – Ibrahim Jun 21 '20 at 16:48
I tried it and I got this. Call to undefined method App\Http\Middleware\School::middleware() @A.A – Ibrahim Jun 21 '20 at 17:00

score 0 · Answer 2 · answered Jun 21 '20 at 16:55

0

You need to redirect the user again in the else of hasAnyRole function:

public function handle($request, Closure $next)
{
    if (Auth::check()) {
        $user = Auth::user();
        if ($user->hasAnyRole('school')) {
            return $next($request);
        } else {
            // HERE
            return redirect('login'); 
        }
    } else {
        return redirect('login'); 
    }
}

answered Jun 21 '20 at 16:55

Giacomo M

4,450
7
28
57

I tried again to redirect them to the other middleware, and still the same. ``` if (!Auth::check()) { return redirect()->route('login'); } $user = Auth::user(); if($user->hasAnyRole('school')) { return $next($request); } $destinations = ['student','teacher']; foreach($destinations as $destination){ return redirect(route( $user->hasAnyrole($destination))); }``` – Ibrahim Jun 21 '20 at 17:23

This page isn’t working localhost redirected you too many times. Try clearing your cookies. ERR_TOO_MANY_REDIRECTS

2 Answers2

Linked