Skip to content

fix(ostool-server): extend session ttl to 30 seconds - #166

Closed
ZCShou wants to merge 1 commit into
mainfrom
fix/session-ttl-30s
Closed

fix(ostool-server): extend session ttl to 30 seconds#166
ZCShou wants to merge 1 commit into
mainfrom
fix/session-ttl-30s

Conversation

@ZCShou

@ZCShou ZCShou commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@mai-team-app mai-team-app Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

本 PR 将 ostool-server 的固定会话 TTL 从 10 秒调整为 30 秒,并把既有单元测试收紧为精确断言 30 秒;这可为串口输出和传输期间的续租提供更充足余量。

影响评估:改动只作用于会话创建和 heartbeat 续期共用的 SESSION_TTL,因此会改变开发板占用、过期清理及依赖 lease_expires_at 的 API 可见租约时长;未改变会话状态机、API 数据模型或串口处理流程,范围看起来是隔离且一致的。已对照既有会话清理、heartbeat 和 WebSocket 刷新调用路径。

验证:GitHub Actions 的两个 check (stable, x86_64-unknown-linux-gnu) 均为 success;未发现失败 CI。已运行审查辅助脚本及其 7 项测试,并验证工作区 HEAD 为 8d791a61d39190dcb3ab9322f3561bb0adb85834。本地 cargo fmt --checkcargo clippy --manifest-path ostool-server/Cargo.toml --all-features -- -D warningscargo test --manifest-path ostool-server/Cargo.toml --all-features 均因环境缺少 cargo(exit 127)无法执行,非代码失败。

此前没有评审评论或 PR 讨论评论需要处理。已检索相近会话生命周期 PR(如 #145#77#67);没有发现与本变更冲突或重复的未解决问题。未发现剩余缺陷或测试缺口。

Powered by gpt-5.6-terra

@ZCShou ZCShou closed this Aug 3, 2026
@YanLien
YanLien deleted the fix/session-ttl-30s branch August 6, 2026 06:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants