mirror of
https://gitee.com/xmz7891/ykxmz.git
synced 2026-10-18 12:30:44 +00:00
长轮询客户端超时 120 -> 180 秒(实测出现 126.6 秒的长尾)
relay-stability 跑了 30 次真实长轮询,第 1 次耗时 126604ms:
服务端 hold 是 60 秒
剩下 66 秒全是网络停顿(4G 抖动 + 重传退避)
而客户端超时写的是 120000ms
-> 126.6 > 120,那次会被**误判成失败**,尽管服务端已经正常返回
其余 29 次都是 61~62 秒,很正常。不是 hold 太长,是网络尾巴太长。
为什么放宽超时而不是缩短 hold:
A. hold 缩到 45 秒、超时保持 120 -> 最坏 111 秒,安全
但请求数多 33%,而每次请求都要吃隧道 1~2 秒开销
B. hold 保持 60 秒、超时放宽到 180 -> 最坏 180 秒,安全
选 B —— 这条链路的根本问题是丢包抖动,**请求越少越不容易撞上**。
放宽超时只是"别把慢但成功的请求冤枉掉",不增加任何请求。
客户端已有"连续失败 3 次才判离线"(OFFLINE_AFTER=3),
所以偶尔一次长等待不会让界面闪"已断开"。
超时余量的演进(全是实测逼出来的):
90 秒 -> 实测 93589ms 不够
120 秒 -> 实测 126604ms 不够
180 秒 -> 60 hold + 120 余量
两边同步改:
windows/lanremote/relay.py POLL_TIMEOUT 120 -> 180
android .../RelayTransport.java POLL_TIMEOUT_MS 120000 -> 180000
(Java 侧确认 setConnectTimeout 另有 15 秒上限、setReadTimeout 吃这个值,
没有别的超时会先把请求掐掉。)
This commit is contained in:
@@ -34,10 +34,15 @@ import urllib.request
|
||||
# 长轮询单次请求的超时。
|
||||
# 必须显著大于服务端的 hold 时间(现在 60 秒),否则会在服务端
|
||||
# 正常返回之前自己先超时 —— 那就变成"每次轮询都失败"了。
|
||||
# 120 = 60(hold) + 60(网络余量)。
|
||||
# 为什么余量给这么大:实测出现过一次 93589ms 的长尾
|
||||
# (4G 丢包触发 TCP 重传退避),原来设 90 秒会把它误判成失败。
|
||||
POLL_TIMEOUT = 120.0
|
||||
# 180 = 60(hold) + 120(网络余量)。
|
||||
# 余量一直在加大,因为实测的长尾一直在刷新:
|
||||
# 90 秒 -> 实测 93589ms 不够
|
||||
# 120 秒 -> 实测 126604ms 不够(30 次里出现 1 次,其余都是 61~62 秒)
|
||||
# 4G 丢包触发的重传退避能拖出一分钟以上的停顿,这是链路特性。
|
||||
# 放宽超时只是"别把慢但成功的请求冤枉掉",不增加请求数。
|
||||
# 而且客户端有"连续失败 3 次才判离线"的保护,
|
||||
# 偶尔一次长等待不会让界面闪"已断开"。
|
||||
POLL_TIMEOUT = 180.0
|
||||
# 单次请求体上限(一个 256KB 分块 Base64 后约 350KB,给足余量)
|
||||
MAX_BODY = 8 * 1024 * 1024
|
||||
# 网络断了之后的退避
|
||||
@@ -243,12 +248,14 @@ class RelayLink:
|
||||
"""开始连中转。
|
||||
|
||||
url 支持用**分号**分隔多个地址,按顺序尝试:
|
||||
http://192.168.2.156:8080;https://relay.xxmxxs.top
|
||||
http://192.168.1.10:8080;https://aaa.bbb.com
|
||||
|
||||
为什么需要多个:
|
||||
* 电脑和随身 WiFi 常在同一局域网 —— 直连局域网中转更快更稳,
|
||||
* 电脑和中转常在同一局域网 —— 直连局域网中转更快更稳,
|
||||
而且地址永远不变(不用管隧道地址变来变去)
|
||||
* 电脑在外面时才走隧道
|
||||
|
||||
注意:**地址由用户自己填**,代码里不内置任何默认中转地址。
|
||||
"""
|
||||
candidates = []
|
||||
for piece in (url or "").replace(",", ";").split(";"):
|
||||
|
||||
Reference in New Issue
Block a user