视频号流式下载端点：两条已知的非阻塞缺陷（gate 第 6 轮，随 PR #10 一并接受） #11

Activity

1. zj1123581321 commented on Sep 23, 2026

   

   zj1123581321

   on Sep 23, 2026

   CollaboratorAuthor

   More actions

   分诊复核（2026-09-23，pi 主脑在 cb1e2b9 实测）

   两条缺陷都已在最新主干修复，本 issue 描述的是 PR #10 时期的形态（_iter_decrypted 那个函数如今已不存在）。实测证据：

   一、首窗 CDN 建连失败 → 已不再逃逸成 500

   # 探针：monkeypatch app.api.stream.open_cdn_stream 抛 httpx.ConnectError，再打端点
   GET /api/stream/wechat_channels/AOzokRxWHz
   → 502  application/json  {"detail":"upstream closed before range complete"}
   → open_cdn_stream 被调用 3 次（收成 UpstreamDisconnected 后按同窗重试）
   

   机理：_read_window 的 except _UPSTREAM_FAIL（httpx.TransportError 在内）+ 端点侧 except UpstreamDisconnected → _json_http_error(502, ...)。
   修复出处：ef9af01 fix(wechat_channels): 窗口 ConnectTimeout 纳入重试并去掉泄漏的 AsyncClient（#16） 与 8373dd1（分窗全读）。

   二、重复构造 httpx client → 已删

   $ grep -n "httpx.AsyncClient(" app/api/stream.py
   89:    client: Optional[httpx.AsyncClient] = httpx.AsyncClient(
   

   只剩一处构造（ef9af01 删掉了泄漏的那个）；所有权由 test_pre_response_cancel_during_cdn_open_closes_client 锁住。

   结论：缺陷不可复现 —— 本 issue 的处置是关单，不派修复卡。
   差距只在「回归锁」：这两条目前没有专门的锁测试（既有测试覆盖的是首窗 CDN 500 与后窗 ConnectTimeout），已派补锁卡覆盖这两个缺口，落地后关单。
2. zj1123581321

   added a commit that references this issue on Sep 23, 2026

   Merge pull request #36 from zlxlabs/card/MediaResolverAPI-stream-lock…

   ...

   c5cec29
3. zj1123581321 commented on Sep 23, 2026

   

   zj1123581321

   on Sep 23, 2026

   CollaboratorAuthor

   More actions

   关单（2026-09-23，PR #36 合入 master c5cec29）

   补锁落地，缺陷锁死：

   • tests/test_stream_boundary_locks.py::test_first_window_connect_error_retries_and_returns_502_json —— 首窗 CDN 传输层建连失败：502、!= 500、application/json、open_cdn_stream 调用 ≥ 2（证明已被收进可重试链路而不是单次逃逸）。
   • #11-2 重复构造 client：master 上 grep -n "httpx.AsyncClient(" app/api/stream.py 仅剩第 89 行一处；所有权由既有 test_pre_response_cancel_during_cdn_open_closes_client 锁住，本轮未再加锁。
   • 红验（最小注入、单行还原）：把 _read_window 的 except _UPSTREAM_FAIL 收窄成 except UpstreamDisconnected → 新用例转红 assert 500 == 502；还原后全绿。

   测试命令：uv run --extra dev python -m pytest tests/test_stream_boundary_locks.py -q → 2 passed（在合入后的 master 上跑）。
4. zj1123581321

   closed this as completedon Sep 23, 2026