52 lines
3.5 KiB
Markdown
52 lines
3.5 KiB
Markdown
|
# 7-50 ONLYOFFICE session.current 诊断工具缺少 resource scope 过滤
|
|||
|
|
|
|||
|
|
## 状态
|
|||
|
|
|
|||
|
|
- 状态：done
|
|||
|
|
- Owner：07-ai / OnlyOffice live bridge / Hermes tools
|
|||
|
|
- 发现时间：2026-06-01
|
|||
|
|
|
|||
|
|
## 现象
|
|||
|
|
|
|||
|
|
`mnote.onlyoffice.session.current` 是诊断工具，不会执行读写动作；但它在未传 `onlyofficeSessionId` / `bridgeSessionId` 时仍会 fallback 到进程内最近活跃 OnlyOffice bridge session，并返回 `sessionId`、`documentId`、`assetId`、`fileType`、pending command/result 计数等元数据。
|
|||
|
|
|
|||
|
|
这与读写工具已经要求 explicit session + `aiAccessScope.allowedResourceIds` 的收口方向不完全一致。若 Page AI 当前 target=A，但进程内最近活跃 Office session 属于 resource=B，模型可能通过 `session.current` 看到 B 的 session 元数据。
|
|||
|
|
|
|||
|
|
## 证据
|
|||
|
|
|
|||
|
|
- `rust/crates/mnote-web/src/hermes_tools/onlyoffice_live.rs` 中 `session_current()` 允许缺 session id 时使用 `current_session_info()`。
|
|||
|
|
- `rust/crates/mnote-web/src/routes/onlyoffice_bridge.rs` 中 `current_session_info()` 返回全局最近活跃 session。
|
|||
|
|
- Reasonix 只读复核 `reasonix-2026-05-31T16-38-36-472Z-63f3a41c` 判断：这不是写绕过，但属于低级信息泄漏风险；建议 `session_current` 也加入 scope 校验或过滤返回字段。
|
|||
|
|
|
|||
|
|
## 影响
|
|||
|
|
|
|||
|
|
- 不会直接写入或读取文档正文，因此风险低于 `7-45` / `7-46`。
|
|||
|
|
- 可能暴露非当前 target resource 的 Office session 元数据，给后续工具调用或模型选择目标带来混淆。
|
|||
|
|
- 与 OnlyOffice live bridge 的最小权限口径不一致。
|
|||
|
|
|
|||
|
|
## 修复建议
|
|||
|
|
|
|||
|
|
- `session_current` 若传 explicit session id，应校验该 session 对应的 `sessionId` / `documentId` / `assetId` / `resource:office:{documentId}:{assetId}` 是否在 `aiAccessScope.allowedResourceIds` 中。
|
|||
|
|
- `session_current` 若未传 explicit session id，不应返回全局最近活跃 session；可以改为返回 `mnote_onlyoffice_session_explicit_required`，或只在 debug/admin 边界允许。
|
|||
|
|
- 如果保留诊断 fallback，至少过滤 `documentId` / `assetId` 等跨 resource 元数据，并在 manifest 标注只用于诊断。
|
|||
|
|
|
|||
|
|
## 本轮处理
|
|||
|
|
|
|||
|
|
- 2026-06-01：`mnote.onlyoffice.session.current` 已改为复用 `resolve_explicit_session_id()` 与 `ensure_onlyoffice_resource_scope_allowed()`。
|
|||
|
|
- 已删除 `onlyoffice_bridge::current_session_info()` 全局最近 session fallback，避免诊断工具继续返回非当前 target 的 session 元数据。
|
|||
|
|
- `manifest.rs` 中 `mnote.onlyoffice.session.current` 已复用 OnlyOffice live 工具 schema，要求 `aiAccessScope.allowedResourceIds`，并通过 `anyOf` 要求 `onlyofficeSessionId` 或 `bridgeSessionId`。
|
|||
|
|
- `scripts/task515-onlyoffice-live-scope-http-smoke.js` 已扩展覆盖 `session.current` 缺 explicit session 返回 400、scope 不匹配返回 403、授权 scope 返回 session 元数据。
|
|||
|
|
|
|||
|
|
## 验收
|
|||
|
|
|
|||
|
|
- [x] Rust 单测覆盖 target=A 的 `aiAccessScope.allowedResourceIds` 不能通过 `session.current` 获取 resource=B 的 session 元数据。
|
|||
|
|
- [x] 缺 explicit session id 的 `session.current` 不再返回全局最近 session。
|
|||
|
|
- [x] `task515` HTTP smoke 覆盖 `session.current` scope 边界。
|
|||
|
|
|
|||
|
|
验证：
|
|||
|
|
|
|||
|
|
- `cargo test --manifest-path rust/Cargo.toml -p mnote-web --lib hermes_tools_onlyoffice_session_current -- --test-threads=1`
|
|||
|
|
- `cargo test --manifest-path rust/Cargo.toml -p mnote-web --lib hermes_tools_manifest_describes_onlyoffice_live_scope -- --test-threads=1`
|
|||
|
|
- `node --check scripts/task515-onlyoffice-live-scope-http-smoke.js`
|
|||
|
|
- `node scripts/task515-onlyoffice-live-scope-http-smoke.js`
|