Skip to content

fix(network): preserve request metadata and bounded response evidence - #728

Closed
lanyue-llk wants to merge 1 commit into
codex/browser-engineering-reliabilityfrom
codex/network-response-evidence
Closed

lanyue-llk wants to merge 1 commit into
codex/browser-engineering-reliabilityfrom
codex/network-response-evidence

Conversation

@lanyue-llk

@lanyue-llk lanyue-llk commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

请求已成功发出,但协议记录缺少实际 Referer 等请求头,导致网络验收无法观察真实行为;重定向和页面切换也可能让请求元数据或响应证据失真。

修复机制

从实际传输结果生成协议观察记录;共享重定向方法与请求体规则。响应体通过有界所有者统一维护存储和索引,按会话启用状态、单条/总量预算及导航生命周期决定可见性和保留。

实际请求头与网络记录

重定向记录

导航后的响应证据保留

以上图片为原报告中的机制示意,非本 PR 的实测截图,也不代表本 PR 已通过验收。来源:五方法报告解读。图片链接固定到证据提交,避免报告更新改变 PR 内容。

对应验收

历史 1.1.9 失败 case:339、676

这些编号描述验收责任,不代表已证明本 PR 独立修复。原大 PR 的历史 90/90 不作为本 PR 的测试结果。

功能回归覆盖:显式重定向预期矩阵、实际请求头、导航保留、会话隔离、禁用与撤销、空/超限正文、替换/删除/淘汰索引一致性。

真实 HTTP POST→302→307 场景以服务端观察作为独立判据,逐跳验证方法、请求体、状态,并用 CDP ExtraInfo 核对实际请求头。同一外部场景实测:Chromium 153 和网络分区通过,main@de109 的两次重定向错误报告 POST 而失败。先前把普通请求事件 Referer 与线上头等同的测试断言已确认不成立,不再作为红测证据;修正后的 Rust 测试由新 HEAD 的 CI 验证。

验证项 范围与状态
当前 HEAD ad2d1083
直接基线 #725 · f13e9f42;其 main 基线为 72413280
格式 / Clippy / 全量 Rust 测试 本 PR CI 验证当前 HEAD
历史真实任务对照 main@de109 完整基线:55/90;原 1.1.9 的 34 个失败均仍失败
本分区真实任务 尚未重复执行;上列 case 是明确验收责任,不宣称已经独立通过

后续真实任务仅针对历史失败集合,保留固定动作与评分规则。功能验收范围为原 1.1.9 的 34 题加 1.1.8 已失败的 386,共 35 题;历史 67 的传输超时另存证据,不归因本 PR 修复。原 11 类分组与原测试改名/替代对应可供审查。

范围与依赖

从 #640 拆出,原 PR 和原分支保留。本 PR 以工程前置 #725 的分支为目标,因此 diff 只显示本功能分区,其他五个功能分区不是当前代码依赖。已随前置 rebase 到 main@72413280。每个分区独立触发 CI;前置合入后可再将本 PR rebase/retarget 到 main。验收发现功能依赖时,先调整边界或明确依赖,再确认 case 归属。

完整对应关系:拆分 PR 索引、逐 case 历史失败及归属。

@lanyue-llk

Copy link
Copy Markdown
Contributor Author

由 #737 替代:基于 main@2793b2407,PR 作者和所有新增提交的 GitHub author/committer 已核验关联 lanyue-llk。原分支、提交和本 PR 的讨论保留;这里只关闭被替代的 PR,不删除分支。新的 CI 与逐 case 验收记录归属替代 PR,旧版本测试结果不冒认为新版本结果。

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