laravelマルチ認証での未ログイン時にリダイレクト先を分ける

Question

Laravelでマルチ認証を使用しています。
admin、userでそれぞれ、認証が必要なページの場合にリダイレクトを設定したいです

認証が必要なページのコントローラーへ

    public function __construct()
    {
        $this->middleware('auth:admin');
    }

のような形で指定しています。
auth.phpにもガードとしてadminを設定しています。

未ログイン時にログイン認証が必要なページにアクセスした時のそれぞれのログインページへのリダイレクト先を指定するために
app/Http/Middleware/Authenticate.phpで

    protected function redirectTo($request)
    {
        ルーティングに応じて未ログイン時のリダイレクト先を振り分ける
        if (!$request->expectsJson()) {
            if (Route::is('admin.order')) {
                return route($this->admin_route);
            } elseif (Route::is('admin.product')) {
                return route($this->admin_route);
            } elseif (Route::is('user')) {
                return route($this->user_route);
            } elseif (Route::is('user.order')) {
                return route($this->user_route);
            }
        }
    }

などとして、振り分けているのですが、
この場合だと、ルートごとに指定しているので、
コントローラーでadminの認証が必要だった場合などは
ルートに関わらず、すべてadmin.loginへリダイレクト、
userの認証が必要だった場合はuser.loginへリダイレクトという形にしたいのですが、
こういった場合は、ここで指定をどのようにかえればよいのでしょうか？
また、middlewareを作成するほうが良いのでしょうか？

よろしくお願いいたします

Answer 1

このミドルウェアにはパラメタでどのガードを使うつもりだったのかが渡されているのですが、redirectToには渡ってこないので惜しい感じなのでそのあたりを改造するのはどうでしょうか。

\App\Http\Middleware\Authenticateの親クラスの\Illuminate\Auth\Middleware\Authenticateを見ると以下のようになっています。

   /**
     * Handle an unauthenticated user.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  array  $guards
     * @return void
     *
     * @throws \Illuminate\Auth\AuthenticationException
     */
    protected function unauthenticated($request, array $guards)
    {
        throw new AuthenticationException(
            'Unauthenticated.', $guards, $this->redirectTo($request)
        );
    }

    /**
     * Get the path the user should be redirected to when they are not authenticated.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return string|null
     */
    protected function redirectTo($request)
    {
        //
    }

つまりredirectToの手前のunauthenticatedまでは$guardsパラメタが渡ってきているので、\App\Http\Middleware\Authenticate側で上書きしてこんな感じにします。

    protected function unauthenticated($request, array $guards)
    {
        throw new AuthenticationException(
            'Unauthenticated.', $guards, $this->redirectToByGuards($request, $guards)
        );
    }

    protected function redirectToByGuards($request, $guards) {
        if (!$request->expectsJson()) {
            if ($guards[0] === 'admin') {
                return route($this->admin_route);
            } else {
                return route($this->user_route);
            }
        }
    }

Comments

[creative_09]

ありがとうございます。
guardが渡っている部分での上書きで処理するのですね。
大変、お手数なのですが
redirectToByGuardメソッドの中身の一例をお願いできないでしょうか？

また、ここまでの流れを組んでいる場合に、ミドルウェアでたとえば、auth_adminなどを作成し、そちらでリダイレクトした場合も違和感のない実装となりえるのでしょうか？

[crhg]

中身の例は回答を更新しました。

後の方の質問についてはどんなミドルウェアを作るつもりなのかがわからないとなんともいえません。

[creative_09]

参考の回答ありがとうございます。
\App\Http\Middleware\Authenticateへ上記のコードを追記し、
コントローラー側へは
public function __construct()
{
$this->middleware('auth:admin');
}
と記述し、実行してみたところ、
Class 'App\Http\Middleware\AuthenticationException' not found
となりエラーとなります。

後者の質問につきましては
コントローラーのconstructに
$this->middleware('auth_admin');
と記述し
karnel.phpに
'auth_admin' => \App\Http\Middleware\AuthAdmin::class
を記述
Middlewareとして、AuthAdmin.phpを作成し、
public function handle($request, Closure $next)
{
if (Auth::guard('admin')->check()) {
return $next($request);
}

//管理者ログインページへ遷移する
return redirect()->route('admin.login');
}
としました。
この流れで実現はできているのですが、提示いただいたものをやってみたいのと、
おっしゃっていた、流れ的にガードを使おうとしていたものが親で止まっているとあったので、どちらが良い流れなのかも気になっています。
どちらにしても、最初に提示したコードのルートによってリダイレクト指定していたものも同時に使いたいのです。
エラーを直してどちらも検証してみたいのでよろしくお願いいたします

[crhg]

`AuthenticationException`は`use`する必要があります。

use Illuminate\Auth\AuthenticationException;

を追加してください。

[creative_09]

ありがとうございます。
無事に思った通りの動作ができました。
ありがとうございます！！
この動作のあとに
protected function redirectTo($request)を通ることになるのであればその後でルートで指定したリダイレクトも可能ということですね

[creative_09]

なぜ低評価がついているのかはわかりませんが、そんな質問に完璧な答えをありがとうございました。
ミドルウェアをあらたに設置するよりもスマートに思います。