Skip to content

fix(platform): 隧道不再强制单协议族,改用 happy-eyeballs 自动选路#394

Merged
deepcoldy merged 3 commits into
masterfrom
fix/tunnel-no-force-family
Jul 7, 2026
Merged

fix(platform): 隧道不再强制单协议族,改用 happy-eyeballs 自动选路#394
deepcoldy merged 3 commits into
masterfrom
fix/tunnel-no-force-family

Conversation

@sensuossss

Copy link
Copy Markdown
Collaborator

问题

botmux restart 后概率性出现 dashboard 显示「平台未授权」、平台侧机器显示离线。

根因:tunnel-client.ts 会把 platform.json 里的 ipFamily 透传给所有 WebSocket 构造器,强制只用单一协议族。当该协议族路由有 ECMP 黑洞时(实测 ~35% 丢包率),冷启动时控制连接连不上平台。更糟的是,某次侥幸连上后会把 ipFamily 写回绑定文件,锁死在坏路径上。

修复

  • tunnel-client.ts: 控制连接和数据流都不传 family,让 Node 内置 happy-eyeballs (autoSelectFamily) 同时拨 IPv4+IPv6,谁先握手成功用谁
  • 移除 setPlatformIpFamily 运行时落盘:不再因一次侥幸连接把 ipFamily 固化到 platform.json
  • bind.ts: 仍保留多族兜底尝试(bind HTTP 请求需要实际打通),但不再把成功的协议族写入绑定文件
  • binding.ts: 删除 setPlatformIpFamily(无调用方)

影响面

  • 改了 src/platform/ 公共层(tunnel-client、bind、binding),影响所有走平台隧道的机器
  • 好处:所有机器的隧道连接都会自动用 happy-eyeballs 选最优路径,不再被单协议族锁死
  • 风险:低——不传 family 只是回退到 Node 默认行为(happy-eyeballs),比强制单族 strictly 更优

测试验证

  • pnpm build ✅
  • pnpm vitest run test/platform-bind-ip-family.test.ts test/tunnel-client-ip-family.test.ts ✅ (8/8 passed)
  • 本机实测:清除 platform.json 里的 ipFamily: 4 后,pnpm daemon:restart 隧道秒连,平台侧机器状态恢复在线

之前 tunnel-client 会把 binding 里的 ipFamily 透传给所有 WebSocket
构造器(family: 4|6),强制只用 IPv4 或 IPv6。当某条协议族路由
有 ECMP 黑洞时(实测 ~35% 丢包率),每次 restart 冷启动都有概率
连不上平台,导致 dashboard 显示「未授权」、平台侧机器离线。

修复:
- tunnel-client.ts: 控制连接和数据流都不传 family,让 Node 内置
  happy-eyeballs (autoSelectFamily) 同时拨 IPv4+IPv6,谁先到用谁
- 移除 setPlatformIpFamily 运行时落盘:不再因一次侥幸连接把
  ipFamily 写回 platform.json 锁死坏路径
- bind.ts: 仍保留多族兜底尝试(bind HTTP 请求需要实际打通),
  但不再把成功的协议族写入绑定文件
- binding.ts: 删除 setPlatformIpFamily(无调用方)

测试:更新 platform-bind-ip-family.test.ts 和
tunnel-client-ip-family.test.ts 验证新行为。
@sensuossss sensuossss requested a review from deepcoldy as a code owner July 7, 2026 09:00
deepcoldy added 2 commits July 7, 2026 09:38
family-flip 逻辑移除后,connectFails 只剩写入(自增/清零)无人读取,
且注释仍称其「驱动 backoff + 协议族翻转」——两者皆已不成立。删除该变量
并更正注释,重连退避由独立的 backoff 变量驱动。
隧道改用 happy-eyeballs 后运行期已不再读取该字段、bind 也不落盘,
旧注释仍称「固定走该协议族」会误导。标注 @deprecated 并说明仅为
兼容旧 platform.json 保留解析。(Codex 二轮 review P3)
@deepcoldy deepcoldy merged commit 3f8c2c2 into master Jul 7, 2026
1 check passed
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.

2 participants