Laravel自定义Login控制器Throttle失效及302跳转问题求助

咱们逐个解决你的两个问题：

1. 登录失败超过3次未触发限流机制 #

你的代码已经引入了ThrottlesLogins trait，但限流没触发的核心原因是登录失败时没有记录尝试次数，同时需要确认限流时间的参数是否匹配Laravel的预期：

问题分析 #

• ThrottlesLogins trait依赖incrementLoginAttempts()方法来记录每次失败的登录尝试，你的代码里在登录失败的分支中没有调用这个方法，所以RateLimiter永远不会累计失败次数，自然不会触发限流。
• 你重写了hasTooManyLoginAttempts()方法，虽然参数传递没问题，但默认的trait方法已经能满足需求，除非有特殊定制，建议优先使用默认实现。

解决方案 #

步骤1：在登录失败分支添加尝试次数累加

在两个登录失败的else块中，调用$this->incrementLoginAttempts($request)：

// 邮箱登录失败分支
else {
    $this->incrementLoginAttempts($request); // 新增这一行
    return redirect()->route('customer-login-page')->with('error', 'Your credentials do not match');
}

// 手机号/联系人登录失败分支
else {
    $this->incrementLoginAttempts($request); // 新增这一行
    return redirect()->route('customer-login-page')->with('error', 'Your credentials do not match');
}

步骤2：（可选）修正lockoutTime的单位（如果需要）

ThrottlesLogins中$lockoutTime的默认单位是分钟，如果你想设置5分钟锁定期，直接写：

protected $lockoutTime = 5; // 5分钟，替代原来的300

如果你坚持用秒作为单位，需要确保hasTooManyLoginAttempts()方法中传递的是正确的秒数，不过建议遵循框架默认的单位规范，避免混淆。

步骤3：（可选）移除自定义的hasTooManyLoginAttempts()方法

默认的trait方法已经实现了正确的逻辑：

// 这是ThrottlesLogins trait的默认实现，足够满足你的需求
protected function hasTooManyLoginAttempts(Request $request)
{
    return $this->limiter()->tooManyAttempts(
        $this->throttleKey($request),
        $this->maxLoginAttempts,
        $this->lockoutTime * 60 // 把分钟转成秒
    );
}

如果没有特殊定制需求，直接删除你重写的这个方法即可，减少不必要的自定义代码。

2. 登录成功返回302 FOUND状态码 #

这其实是正常的HTTP行为，不需要修复：

• Laravel的redirect()方法默认返回302临时重定向状态码，这符合HTTP规范：登录成功后的跳转是临时的，用户下次访问登录页依然需要重新登录，不是永久跳转。
• 如果你确实需要返回301永久重定向（不推荐这种场景），可以修改跳转代码为：

return redirect()->route('front')->permanent();

但通常登录成功后的跳转用302是完全合理的，所以这个“问题”本质上是符合预期的现象。

内容的提问来源于stack exchange，提问作者Sai Krishna