nixevol/NixMsg
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[R3-05][high] JS SDK 传输失败后发送队列不再继续 #69

New Issue
Closed
opened 2026-09-30 17:18:25 +08:00 by nixevol · 2 comments
nixevol commented 2026-09-30 17:18:25 +08:00
Owner
Copy Link
Copy Source

现象

基线 origin/main 0c9b459。publishUp 抛出的不是协议 APIError 时，代码会换新 rid 并清掉 inflight，然后 return。没有调用 drainSendQueue。连接若没有因此断开、hello 也不会再走一遍，send() 的 Promise 一直不结束。rate_limited 路径会 schedule 再泵，传输错误这条没有。

位置

  • sdk/js/src/client.ts：发送 catch 里 !(e instanceof APIError) 的分支
  • 对照：同文件 rate_limited 分支的 schedule(() => void this.drainSendQueue(), wait)，以及 doHello 里的唤醒

方案

传输失败且未 stopReconnect 时，换新 rid 后安排 drainSendQueue（可立即或按现有退避）。stopReconnect 时结束这次 send 并返回错误。不要只 return。

补测试：假传输 publishUp 失败一次后，队列会再次发送，且 rid 已变、id / body / send_at_ms 保留。

约束

只改 sdk/js/。不要改 Go / Python / Java。不要对同一段等待再乘一次抖动。偏差节 ### 复审修复 R3-05。

## 现象 基线 `origin/main` `0c9b459`。`publishUp` 抛出的不是协议 `APIError` 时，代码会换新 `rid` 并清掉 inflight，然后 `return`。没有调用 `drainSendQueue`。连接若没有因此断开、hello 也不会再走一遍，`send()` 的 Promise 一直不结束。`rate_limited` 路径会 `schedule` 再泵，传输错误这条没有。 ## 位置 - `sdk/js/src/client.ts`：发送 catch 里 `!(e instanceof APIError)` 的分支 - 对照：同文件 `rate_limited` 分支的 `schedule(() => void this.drainSendQueue(), wait)`，以及 `doHello` 里的唤醒 ## 方案 传输失败且未 `stopReconnect` 时，换新 `rid` 后安排 `drainSendQueue`（可立即或按现有退避）。`stopReconnect` 时结束这次 `send` 并返回错误。不要只 `return`。 补测试：假传输 `publishUp` 失败一次后，队列会再次发送，且 `rid` 已变、`id` / `body` / `send_at_ms` 保留。 ## 约束 只改 `sdk/js/`。不要改 Go / Python / Java。不要对同一段等待再乘一次抖动。偏差节 `### 复审修复 R3-05`。
nixevol commented 2026-09-30 17:29:06 +08:00
Author
Owner
Copy Link
Copy Source

R3-05 ????

??:feat/fix-r3-69-js-sdk
HEAD:7f308001927e9437b5497daa2c8562b1fcc6a920

??

  • sdk/js/src/client.ts:publishUp ????? APIError ?? stopReconnect ?,?? rid ??? drainSendQueue;? stopReconnect ? finishSendErr?
  • ??:????????????? rid ??,id/??/send_at_ms ???
  • docs/DEVIATIONS.md ??????? R3-05??

??

? sdk/js ? npm test:4 files / 49 tests ?????

???,??? main?

## R3-05 ???? ??:`feat/fix-r3-69-js-sdk` HEAD:`7f308001927e9437b5497daa2c8562b1fcc6a920` ### ?? - `sdk/js/src/client.ts`:`publishUp` ????? `APIError` ?? `stopReconnect` ?,?? `rid` ??? `drainSendQueue`;? `stopReconnect` ? `finishSendErr`? - ??:????????????? `rid` ??,`id`/??/`send_at_ms` ??? - `docs/DEVIATIONS.md` ??????? R3-05?? ### ?? ? `sdk/js` ? `npm test`:4 files / 49 tests ????? ???,??? `main`?
nixevol commented 2026-09-30 19:21:09 +08:00
Author
Owner
Copy Link
Copy Source

已合入 main：08c08de (#69)；当前 origin/main HEAD 2d1dfd7。

已合入 main：`08c08de` (#69)；当前 origin/main HEAD `2d1dfd7`。
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
P0-critical
严重：可导致全站不可用、数据丢失或无需凭据的攻击
 P1-high
高：核心功能错误或明显安全问题
 P2-medium
中：功能偏差、健壮性或一致性问题
 P3-low
低：体验、质量或小问题
 lane/admin
工作线：管理后端
 lane/broker
工作线：broker（internal/broker）
 lane/identity
工作线：身份、群、在线与认证基础
 lane/listener
工作线：监听与 HTTP
 lane/message
工作线：消息核心
 lane/sdk
工作线：SDK
 lane/store
工作线：存储、配置、命令行与部署
 lane/test-docs
工作线：测试与文档
 lane/web
工作线：后台网页
 review-2026-09-30
2026-09-30 全量复审提出
 tracking
总览与跟踪
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
nixevol (Nixevol)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: nixevol/NixMsg#69