Laravel middleware returning (Trying to get property 'headers' of non-object) error

Question

I'm getting error when I wrap a resource route to my custom middleware

My middleware:

<?php

 namespace App\Http\Middleware;

 use Closure;
 use Auth;

class Officer
{
    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Closure  $next
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        if (Auth::check() && Auth::user()->role == 'title_officer') {
            return $next($request);
        }
        // elseif (Auth::check() && Auth::user()->role == 'agent') {
        //     return redirect('/agent');
        // }
        // else {
        //     return redirect('/customer');
        // }
    }
}

The resource route using the middleware:

Route::resource('new-order', 'BackendController')->middleware('officer');

I'm getting error:

(Trying to get property 'headers' of non-object).

How to fix it ?

score 43 · Accepted Answer · answered Feb 12 '19 at 07:18

In middleware, it is important to handle all cases and return the redirects accordingly or abort.

You do return $next($request); when you want to allow system to continue processing the request ahead.

However, if in case if (Auth::check() && Auth::user()->role == 'title_officer') condition fails, you have not mentioned what system should do.

You can may be abort(404) if you do not want to show the page as available or maybe abort(403) for access forbidden.

public function handle($request, Closure $next)
    {
        if (Auth::check() && Auth::user()->role == 'title_officer') {
            return $next($request);
        }
        abort(403);
    }

But make sure you do not add a case which will make an infinite loop. Please check documentation for more options.

score 3 · Answer 2 · edited Jul 04 '20 at 00:21

3

just use some thing like this in middleware : RedirectIfAuthenticated

public function handle($request, Closure $next, $guard = null)
{
     if (Auth::guard($guard)->check()) {
         // your condition
         return redirect('the route name');
     }
}

edited Jul 04 '20 at 00:21

Ersoy

8,816
6
34
48

answered Jul 03 '20 at 23:21

Android user

65
5

Great! I was missing the redirect helper. – JCarlosR Dec 23 '20 at 06:10

score 1 · Answer 3 · answered Mar 15 '22 at 13:37

1

I had this error. In my controller I had

return route('dashboard')

instead of

return redirect()->route('dashboard')

That should fix it

answered Mar 15 '22 at 13:37

kevlon galloway

11
1

score 0 · Answer 4 · answered Feb 12 '19 at 05:59

0

try adding an else block which returns when your if condition isn't met:

public function handle($request, Closure $next)
    {
        if (Auth::check() && Auth::user()->role == 'title_officer') {
            return $next($request);
        }else{
           return back();
        }
    }

answered Feb 12 '19 at 05:59

Kapitan Teemo

2,144
1
11
25

the missing else block was creating the problem, but now i fixed it. Thanks – MorshedSaif Feb 12 '19 at 07:25

score 0 · Answer 5 · answered Jan 03 '23 at 19:16

0

Check middleware function and you have missed below one_

return $next($request);

so add this line.

answered Jan 03 '23 at 19:16

Limon

21
2

score -1 · Answer 6 · answered Jul 31 '20 at 20:53

-1

Late but.. This happens to me when column in DB was null.

answered Jul 31 '20 at 20:53

sam00vun0c

29
4

Laravel middleware returning (Trying to get property 'headers' of non-object) error

6 Answers6

Linked