Skip to content

docs(security): 分诊表刷新到 56 条现状 + 逐条判定台账 + tracker 反向链接 - #154

Merged
ReSerendipity merged 1 commit into
mainfrom
docs/codeql-triage-56
Sep 23, 2026
Merged

ReSerendipity merged 1 commit into
mainfrom
docs/codeql-triage-56

Conversation

@ReSerendipity

Copy link
Copy Markdown
Owner

纯文档(2 个 md 文件),无代码改动。现状数字实测:open 分布 36 py/path-injection + 18 py/stack-trace-exposure + 2 py/reflective-xss = 56。

未做的事(明确留给下一阶段)

  • 19 条"已缓解"的 path-injection 没有替你去 dismiss,只在表里给了判定与依据;要收口请单独发一批,理由文案我已写在 §7.2。
  • 17 条无强校验的 path-injection(generation.py 3 / persona_metadata.py 6 / resolve_persona_ref 2 / voxcpm2 design+script 6)与 5 条 settings.py 把 str(exc) 放进响应 —— 本阶段按指示不改代码,只给判定。建议统一复用 persona_manager 的白名单 + realpath 前缀,而不是每处再写一遍 basename()。
  • fix(security): 转义 voicebox / step-audio-editx 成功页的请求可控插值(真反射型 XSS) #153 未合并前,§7 里那 2 条 reflective-xss 仍在告警表里(不是我漏记)。

新增 §7(现状以该节为准,§6 是 09-21 快照):

- 本批 dismiss 16 条(routes/model.py 的 11 条 stack-trace 按 #96 绑 mitigated;
  3 条 blob:/textContent 判误报;tabs.py 相对重定向 1 条;voxcpm2/design.py 去标签
  属规则误用 1 条),每条在 CodeQL 上写了 280 字符内的绑定理由。open 76 → 56(实测分布
  36 path-injection + 18 stack-trace + 2 reflective-xss,后者等 #153 合并消解)。
- 剩余 40 条逐条给判定:19 条已缓解可下批收口;17 条 path-injection 无强校验
  (generation 3 / persona_metadata 6 / resolve_persona_ref 2 / voxcpm2 design+script 6);
  stack-trace 里确认 5 处 settings.py 把 str(exc) 原文放进响应,13 处待读。
- 写明一条口径:html.escape 只防 XSS,不防信息泄露,stack-trace 这族不能拿它交差。
- §7.3 衔接 issue #97(锁集已按"退"路自洽,但 4.52.x 的 16 条公告代价未见 pip-audit
  命中/豁免说明)与 #99(3 处真 XSS 的 incomplete-sanitization 整族已归零,修法未复核)。
- SECURITY_REMEDIATION_TRACKER.md 补指向本表的反向链接(此前该链接依赖已关闭的 #81)。

Signed-off-by: ReSerendipity <zengyangc@outlook.com>
@ReSerendipity
ReSerendipity enabled auto-merge (squash) September 23, 2026 16:14
@ReSerendipity
ReSerendipity merged commit 2433c10 into main Sep 23, 2026
29 checks passed
@ReSerendipity
ReSerendipity deleted the docs/codeql-triage-56 branch September 24, 2026 11:55
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.

1 participant