Include the passport guard when throwing exception

[willrowe]

This passes the guard used for passport when throwing an authentication exception.

When this exception is thrown, the default behavior is to redirect to the login page. This login page is usually associated with the default web guard. If a custom guard is used for Passport, you may want to redirect to a different URL. Including the guard itself in the exception allows the exception handler to detect it and change the redirect.

[taylorotwell]

Tests are failing on this one.

[willrowe]

Sorry, I got caught up with something and haven't been able to get back to this yet. Thanks for marking it as draft.

[hafezdivandari]

The guard on AuthenticationException not always equals passport.guard but depends on the context. For example this exception is thrown on CheckScopes middleware, but it has nothing to do with passport.guard. Actually this only makes sense on AuthorizationController:

// src/Http/Controllers/AuthorizationController.php

protected function promptForLogin(Request $request): never
{
    $request->session()->put('promptedForLogin', true);

-   throw new AuthenticationException;
+   throw new AuthenticationException(guards: isset($this->guard->name) ? [$this->guard->name] : []);
}

Note: SessionGuard is the only implementation of the StatefulGuard and it has name property.

[willrowe]

The guard on AuthenticationException not always equals passport.guard but depends on the context. For example this exception is thrown on CheckScopes middleware, but it has nothing to do with passport.guard. Actually this only makes sense on AuthorizationController:

// src/Http/Controllers/AuthorizationController.php

protected function promptForLogin(Request $request): never
{
    $request->session()->put('promptedForLogin', true);

-   throw new AuthenticationException;
+   throw new AuthenticationException(guards: isset($this->guard->name) ? [$this->guard->name] : []);
}

Note: SessionGuard is the only implementation of the StatefulGuard and it has name property.

Good points, I was trying to make the least amount of changes, but you're right, it would not be accurate. I'm going to rethink the approach.