自定义中间件未遵循Eloquent的update调用，并发请求下登出验证异常

核心问题分析 #

你的代码中查询[1]和更新[2]是两个独立操作，并非原子性执行。当多个并发请求触发中间件时，第二个请求可能在第一个请求完成update前执行查询，从而获取到未更新的logged_out_at为null的记录，导致不符合预期的结果。

可行解决方案 #

1. 原子化更新操作（推荐） #

将查询与更新合并为一个原子数据库操作，利用数据库本身的锁机制保证只有第一个请求能成功更新记录。Laravel的update方法会直接生成原子SQL语句，数据库会自动处理并发冲突：

$updatedCount = Logout::where("user_id", Auth::id())
    ->where("logged_out_at", null)
    ->update($params);

// 只有更新成功的请求（影响行数>0）才执行登出逻辑
if ($updatedCount > 0) {
    return response()->json(['message' => 'Not authorized'], 401);
    // 或重定向到登出路由：return redirect()->route('logout');
}

这种方式无需额外锁，依赖数据库原生的原子操作，性能和可靠性都更优。

2. 数据库行级锁 + 事务 #

通过事务和行级锁，在查询时锁定目标记录，阻止其他并发请求读取或修改该行，直到当前事务完成：

DB::beginTransaction();
try {
    // 使用lockForUpdate获取行级排他锁
    $record = Logout::where("user_id", Auth::id())
        ->where("logged_out_at", null)
        ->lockForUpdate()
        ->first();

    if ($record) {
        $record->update($params);
        DB::commit();
        return response()->json(['message' => 'Not authorized'], 401);
    }
    DB::commit();
} catch (\Exception $e) {
    DB::rollBack();
    // 按需处理异常，比如返回服务器错误
    return response()->json(['message' => 'Server error'], 500);
}

注意：行级锁会增加数据库的开销，高并发场景下需评估性能影响。

3. 重构登出机制设计 #

如果业务允许，可考虑更高效的登出验证方案：

• 使用JWT令牌的黑名单机制：登出时将令牌加入Redis黑名单，中间件验证令牌是否在黑名单内
• 直接操作用户会话：若使用Laravel内置会话，调用Auth::logout()会销毁会话，天然避免并发问题

原尝试方案失效原因 #

• Redis缓存：缓存更新与数据库更新存在一致性延迟，并发请求仍可能在缓存同步前读取旧状态
• 全局数组变量：PHP是多进程模型，不同请求可能运行在不同进程中，全局数组无法跨进程共享，导致状态混乱

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