中间件内重定向至控制器动作时出现无限循环（302错误）

嘿，这个无限循环的坑我之前也踩过！大概率是你的中间件没把重定向的目标路由排除在外，导致每次重定向后又被中间件拦截，再次触发重定向，形成死循环。咱们一步步拆解问题、解决它：

先捋清楚你的代码隐患 #

先看你贴的构造函数部分，这里有两个小问题：

1. 构造函数阶段Laravel的认证系统可能还没完全初始化，直接调用Auth::user()容易出现认证状态判断不准的情况；
2. 你的认证判断逻辑写错了：Auth::user()已经是当前用户实例，不需要再调用->user()，而且guest()是Auth门面的方法，不是用户实例的方法。

不过更核心的无限循环原因，肯定是重定向的目标路由也被这个IP检测中间件覆盖了。

具体解决方案 #

1. 排除重定向目标路由（最关键） #

你得把需要跳转到的那个控制器动作对应的路由，从中间件的作用范围里排除，不然每次跳转都会被中间件抓回来。

• 如果是全局注册的中间件，你可以在中间件里直接判断当前路由：

public function handle($request, Closure $next)
{
    // 假设你的重定向目标路由名称是'ip.blocked'，先排除它
    if ($request->route()->getName() === 'ip.blocked') {
        return $next($request);
    }

    // 你的IP检测逻辑写在这里
    if ($this->userIP && $this->shouldBlock($this->userIP)) {
        // 重定向到目标控制器动作，比如BlockController@show
        return redirect()->action([BlockController::class, 'show']);
    }

    return $next($request);
}

• 或者如果你的中间件是分组注册的，也可以在app/Http/Kernel.php里给中间件加排除规则（部分中间件支持）。

2. 修复构造函数的认证判断逻辑 #

把认证相关的判断移到handle方法里更可靠，而且修正判断逻辑：

public function __construct()
{
    $this->userIP = !empty($_SERVER["HTTP_CF_IPCOUNTRY"]) ? $_SERVER["HTTP_CF_IPCOUNTRY"] : false;
}

public function handle($request, Closure $next)
{
    // 把认证判断放到handle里，此时请求上下文更完整
    $user = Auth::user();
    if (Auth::guest() || ($user && $user->isAnonymous())) {
        $this->auth = null;
    } else {
        $this->auth = $user;
    }

    // ...剩下的IP检测和路由排除逻辑
}

3. 给重定向加明确的终止条件 #

你的IP检测逻辑一定要有清晰的边界：比如用户IP在允许列表里、已经跳转到目标页面，就不要再触发重定向。可以加个日志辅助排查：

use Illuminate\Support\Facades\Log;

public function handle($request, Closure $next)
{
    Log::info("IP中间件触发，当前路由：{$request->route()->getName()}，用户IP国家：{$this->userIP}");
    
    // 路由排除逻辑...
    
    return $next($request);
}

打开日志就能清楚看到中间件是不是一直在循环触发。

快速排查小技巧 #

打开浏览器开发者工具的「网络」面板，看循环的请求记录：如果每次都是同一个URL在302跳转，那100%是路由没排除的问题。

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