【Laravel】ログインユーザーがログインページに飛べてしまう

Question

Auth::routesを使ったのに、ログインユーザーがログインページに飛べてしまいます。
また、逆に飛べるはずのregisterに飛べません。
フレームワークにはそこまで用意されていないのか。
Auth::routesを使ったログインは調べると見つかりますがそれ以降の記事が見つからず。

// RedirectIfAuthenticated

    public function handle($request, Closure $next, $guard = null)
    {
        if (Auth::guard($guard)->check()) {
            // return redirect(RouteServiceProvider::HOME);
            return redirect('/');
        }

        return $next($request);
    }

// web.php

Auth::routes(['register' => false]);
Route::group(['middleware' => ['auth']], function () {
    Route::get('/', 'Dashboard\ViewController@index')->name('dashboard.index');
    Route::get('/register', 'Auth\RegisterController@showRegistrationForm')->name('register');
    Route::post('/register', 'Auth\RegisterController@register');
});

// Handler

    protected function unauthenticated($request, AuthenticationException $exception)
    {
        if ($request->expectsJson()) {
            return response()->json(['error' => 'Unauthenticated.'], 401);
        }

        return redirect()->guest(route('login'));
    }

<?php
// LoginConroller.php

namespace App\Http\Controllers\Auth;

use App\Http\Controllers\Controller;
use App\Http\Requests\LoginRequest;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;

class LoginController extends Controller
{
    /**
     * @return View
     */
    public function showLoginForm() {
        return view('login.index');
    }

    /**
     * @param App\Http\Requests\LoginRequest $request
     */
    public function login(LoginRequest $request) {

        $credentials = $request->only('email', 'password');

        if (Auth::attempt($credentials)) {
            $request->session()->regenerate();
            return redirect('/');
        }

        return back()->withErrors([
            'login_error' => 'メールアドレスかパスワードが間違っています'
        ]);
    }

    public function logout()
    {
        Auth::logout();
        return redirect('/login');
    }
}

<?php

namespace App\Http;

use Illuminate\Foundation\Http\Kernel as HttpKernel;
use Laravel\Sanctum\Http\Middleware\EnsureFrontendRequestsAreStateful;

class Kernel extends HttpKernel
{
    /**
     * The application's global HTTP middleware stack.
     *
     * These middleware are run during every request to your application.
     *
     * @var array
     */
    protected $middleware = [
        // \App\Http\Middleware\TrustHosts::class,
        \App\Http\Middleware\TrustProxies::class,
        \Fruitcake\Cors\HandleCors::class,
        \App\Http\Middleware\CheckForMaintenanceMode::class,
        \Illuminate\Foundation\Http\Middleware\ValidatePostSize::class,
        \App\Http\Middleware\TrimStrings::class,
        \Illuminate\Foundation\Http\Middleware\ConvertEmptyStringsToNull::class,
    ];

    /**
     * The application's route middleware groups.
     *
     * @var array
     */
    protected $middlewareGroups = [
        'web' => [
            \App\Http\Middleware\EncryptCookies::class,
            \Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse::class,
            \Illuminate\Session\Middleware\StartSession::class,
            // \Illuminate\Session\Middleware\AuthenticateSession::class,
            \Illuminate\View\Middleware\ShareErrorsFromSession::class,
            \App\Http\Middleware\VerifyCsrfToken::class,
            \Illuminate\Routing\Middleware\SubstituteBindings::class,
        ],

        'api' => [
            EnsureFrontendRequestsAreStateful::class,
            'throttle:60,1',
            \Illuminate\Routing\Middleware\SubstituteBindings::class,
        ],
    ];

    /**
     * The application's route middleware.
     *
     * These middleware may be assigned to groups or used individually.
     *
     * @var array
     */
    protected $routeMiddleware = [
        'auth' => \App\Http\Middleware\Authenticate::class,
        'auth.basic' => \Illuminate\Auth\Middleware\AuthenticateWithBasicAuth::class,
        'bindings' => \Illuminate\Routing\Middleware\SubstituteBindings::class,
        'cache.headers' => \Illuminate\Http\Middleware\SetCacheHeaders::class,
        'can' => \Illuminate\Auth\Middleware\Authorize::class,
        'guest' => \App\Http\Middleware\RedirectIfAuthenticated::class,
        'password.confirm' => \Illuminate\Auth\Middleware\RequirePassword::class,
        'signed' => \Illuminate\Routing\Middleware\ValidateSignature::class,
        'throttle' => \Illuminate\Routing\Middleware\ThrottleRequests::class,
        'verified' => \Illuminate\Auth\Middleware\EnsureEmailIsVerified::class,
    ];
}

追記
バージョン追加
laravel 7
laravel/ui 2.4

Comments

[Lulucom]

Laravelのバージョンも追記された方が良いと思います。認証周りはバージョンによってかなり違うと思いますので。

> 飛べるはずのregisterに飛べません。

registerに飛べないのは以下の行でregisterを無効にしているためではないでしょうか。

Auth::routes(['register' => false]);

[nakamu]

投稿ありがとうございます。現状他のパスワードリセット画面にも飛べてしまいます。

[Lulucom]

ログインユーザーがパスワードリセット関連の画面に飛べるのは問題ないと思います。

php artisan route:listコマンドの結果と、以下のコードも掲載されると良いと思います。
- App\Http\Kernelクラスの$routeMiddleware
- App\Http\Controllers\Auth\LoginControllerクラスのコンストラクタ

[nakamu]

ログイン済みユーザーがログイン画面に飛べてしまうのをなんとかしたいです！

Answer 1

ログインのアクションに guest ミドルウェアを指定していないためだと思います。
LoginController クラスに以下のようなコンストラクタを追加するとどうでしょうか。

    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->middleware('guest')->except('logout');
    }

RedirectIfAuthenticatedは、App\Http\Kernelクラスの$routeMiddlewareでguestとして定義されています。これをログインのアクションに指定する必要があります。