Jetty项目SessionDataStore中孤儿会话清理机制的问题与修复

问题背景

在Jetty 12.0.5版本中，使用JDBC SessionDataStore时发现了一个关于会话管理的缺陷。具体表现为：当应用运行时，过期的"孤儿会话"(orphaned sessions)不会被自动清理，只有在服务器重启时才会被删除。这个问题影响了使用默认配置的会话存储实现。

技术原理

Jetty的会话管理机制中，AbstractSessionDataStore类负责处理会话数据的存储和清理。其中包含一个重要的功能——定期清理孤儿会话。孤儿会话是指那些在服务器内存中已经不存在，但在持久化存储(如数据库)中仍然存在的会话记录。

清理机制的核心逻辑是：

1. 通过_lastOrphanSweepTime记录上次清理时间
2. 根据_gracePeriodSec(宽限期)和当前时间判断是否需要执行清理
3. 调用cleanOrphans()方法实际执行清理操作

问题根源分析

在原始代码实现中，存在一个逻辑错误：无论是否满足清理条件，_lastOrphanSweepTime都会被更新为当前时间。这导致清理条件now > (_lastOrphanSweepTime + TimeUnit.SECONDS.toMillis(10 * _gracePeriodSec))几乎永远不会成立，因为：

1. 默认的_gracePeriodSec是1小时(3600秒)
2. 默认的清理间隔是10倍的_gracePeriodSec(即10小时)
3. 由于每次检查都会更新时间戳，实际清理间隔被无限延长

解决方案

修复方案是将更新时间戳的操作移到条件判断内部，确保只有在实际执行清理后才更新时间戳。具体修改如下：

if (now > (_lastOrphanSweepTime + TimeUnit.SECONDS.toMillis(10 * _gracePeriodSec)))
{
    try
    {
        if (LOG.isDebugEnabled())
            LOG.debug("Cleaning orphans at {}, last sweep at {}", now, _lastOrphanSweepTime);
               
        cleanOrphans(now - TimeUnit.SECONDS.toMillis(10 * _gracePeriodSec));
    }
    finally
    {
        _lastOrphanSweepTime = now;
    }
}

影响范围

该问题主要影响：

1. 使用JDBC SessionDataStore的场景
2. 采用默认配置(grace period为1小时)的环境
3. 需要长期运行且会话创建频繁的应用

最佳实践建议

对于生产环境中的会话管理，建议：

1. 根据应用特点合理设置_gracePeriodSec值
2. 定期检查会话存储中的记录数量
3. 在升级Jetty版本时验证会话清理功能
4. 对于高并发应用，考虑调整清理频率以避免性能影响

该修复已合并到Jetty代码库中，用户可以通过升级到包含修复的版本来解决此问题。