更新Session值时如何避免竞态条件？

问题根源 #

你的代码逻辑顺序有误：先把session_last_activity更新为当前时间，再用旧的会话时间判断是否过期。当并发请求进来时，每个请求都会读取到过期的旧时间，进而都生成新的some_value并创建数据库记录。

方案1：调整控制器内的逻辑顺序 #

把「判断过期」→「处理some_value」→「更新会话时间」的顺序理顺，同时利用Session的原子性获取逻辑，确保只有第一个请求会生成新值：

public function some_method(Request $request)
{
    // 1. 先读取当前会话的状态，不要先更新时间
    $session_last_activity = session('session_last_activity');
    $current_some_value = session('some_value');

    // 2. 判断会话是否过期
    $is_expired = is_null($session_last_activity) || $session_last_activity < now()->subMinutes(10);

    // 3. 处理some_value：只有当会话过期且session中没有有效值时才生成新值
    if ($is_expired) {
        // 利用Session的get方法回调实现原子性：如果已存在新值就直接用，否则生成并存入
        $some_value = session()->get('some_value', function () {
            return Str::random(30);
        });
        // 确保新值存入Session（如果是回调生成的，这里会自动存入，可省略，但显式写更稳妥）
        session()->put('some_value', $some_value);
    } else {
        $some_value = $current_some_value;
    }

    // 4. 最后更新会话最后活动时间
    session(['session_last_activity' => now()]);

    // 后续数据库操作，使用$some_value
    // ...
}

方案2：用中间件统一处理会话状态 #

把会话过期检查和some_value生成逻辑抽离到中间件，让请求进入控制器前就完成状态同步，结合Laravel Session的默认锁定机制（比如file驱动的会话文件锁），同一个用户的并发请求会被串行处理，从根源避免重复生成：

1. 创建中间件：

php artisan make:middleware RefreshSessionValue

2. 编写中间件逻辑：

<?php

namespace App\Http\Middleware;

use Closure;
use Illuminate\Http\Request;
use Illuminate\Support\Str;

class RefreshSessionValue
{
    public function handle(Request $request, Closure $next)
    {
        $last_activity = session('session_last_activity');
        $is_expired = is_null($last_activity) || $last_activity < now()->subMinutes(10);

        if ($is_expired) {
            // 原子获取或生成新值
            $some_value = session()->get('some_value', function () {
                return Str::random(30);
            });
            session()->put('some_value', $some_value);
        }

        // 更新会话最后活动时间
        session()->put('session_last_activity', now());

        return $next($request);
    }
}

3. 在app/Http/Kernel.php中注册中间件，分配给对应的路由或控制器：

protected $routeMiddleware = [
    // ...
    'refresh.session' => \App\Http\Middleware\RefreshSessionValue::class,
];

4. 控制器中直接使用：

public function some_method(Request $request)
{
    // 直接从Session获取已处理好的some_value
    $some_value = session('some_value');

    // 后续数据库操作
    // ...
}

关键说明 #

Laravel默认的Session驱动（如file）会在请求期间锁定会话文件，同一个用户的并发请求会被串行处理，确保第一个请求完成会话更新后，后续请求能读取到最新的some_value，不会重复生成。如果使用Redis等分布式Session驱动，只要Session配置开启了锁定（Redis驱动默认开启），同样能生效。

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