mirror of
https://gitee.com/xmz7891/ykxmz.git
synced 2026-10-18 14:50:45 +00:00
== 现象 ==
用户报:
* 局域网能搜到电脑,IP/端口/配对码都对,连上后**没有任何反应**
* 走中转「已配对」,但点按钮没反应
局域网那一半已经修过(preferLan)。这次抓到的是**经中转**那一半,
而且比预想的严重得多。
== 证据链(插桩复现出来的)==
被控端日志里直接抛了未捕获异常,线程当场死:
Exception in thread client-relay:
server.py:407 session.send(reply)
relay.py:166 self._link.send_to_client(data)
relay.py:357 self._post("/v1/agent/send", ...)
relay.py:766 raise RelayError(f"请求失败: {last_error}")
lanremote.relay.RelayError: 请求失败: Idle
之后的状态:
* client-relay 线程没了 —— 电脑再也不往手表回任何东西
* relay-link 线程还活着、还在长轮询 —— 中转那边看着「设备在线」
* **手表看到的就是「连上了,但按一下毫无反应」**
写入仓库的测试 _tests/test_relay_local.py 一直是这个形状:
前面几项 PASS、跑到经中转发命令就卡死。之前没人深挖。
== 根因 1:一条 HTTP/1.1 连接被两个线程同时用 ==
_post() 里锁只包住「取连接」那一段:
with self._http_lock:
connection = self._http # 锁到这里就放了
try:
connection.request("POST", ...) # 锁外发请求
于是两个线程会同时用同一条 connection:
* relay-link 线程跑 /v1/agent/poll —— 长轮询,一条请求挂 180 秒
* client-relay 线程同时 POST /v1/agent/send
HTTP/1.1 一条连接同一时刻只能有一个在途请求,第二个线程一进来
就撞上 http.client 状态机,抛 CannotSendRequest("Idle")。
_post 的「重试一次」也救不了:它 _drop_http() 之后,另一个线程
手里还攥着那条旧连接对象在发,照样坏。
修法:
* **发命令用独立连接**(_post 加 dedicated 参数)。
长轮询那条常年被占着,发命令不该去挤它,各走各的。
* 两条连接各自一把锁**锁住整个「发请求 + 读响应」周期**,
不再只锁取连接那一下。
* 普通请求若发现缓存连接正忙(长轮询在途),临时另开一条用完即关。
* _drop_http(dedicated=) 区分清哪条;finally 里一定归还占用
(否则那条连接会被永久标记为忙)。
== 根因 2:回包失败会打死整个会话线程 ==
server.py 的 _client_loop 里:
if reply is not None:
session.send(reply) # <- 不在任何 try 里
...
except OSError: # <- 外层只捕获 OSError
pass
而中转链路上 send 抛的是 RelayError,直接冒出去把线程打死。
**死一次就永久哑**,代价太大。
修法:session.send() 单独兜住所有异常,失败就断开这个会话
(让手表重连),绝不让线程消失。外层也补一个兜底 except。
== 修复效果 ==
修复前:pair OK -> ping OK -> ping 卡死 -> 线程崩溃,永久哑
修复后:pair 21ms -> ping 82ms -> ping 4ms -> ping 6ms,零异常
_tests/test_relay_local.py:**之前完全卡死 -> 现在 21/21 全过**
这个测试之前一直是红的,说明「35 个阶段的回归全绿」这件事
本身就不成立过 —— 某个阶段卡住时看起来像环境问题而不是缺陷。
== 同批修掉验证阶段找到的其它问题 ==
1. **TTL 清理 + 关掉接入开关 = 设备永久锁死**(严重)
register() 用 `device_id in self.devices` 判断老设备,但超时清理
把设备从 devices 和 codes 都删了。电脑关机过夜(必然 > 6 小时)
第二天开机就会被自己的开关挡住,而且**重启也救不回来**
(清 codes 时没落盘,重启后又当孤儿清一遍,自我维系)。
管理员在页面上还看不到这台设备,不知道该放开谁。
修法:状态文件里单独存 knownDevices 设备历史,超时清理绝不动它;
register 改查这份历史;清理后补 _save();delete_device 才真正忘记。
2. **管理页 JS 语法错误 -> 所有按钮失效**(严重)
admin.py 里 ADMIN_JS 是普通三引号字符串,我在 Python 源码里写的
`\n` 被变成真实换行,拼进 JS 就是字符串里的裸换行 ->
整段 <script> 不执行 -> 改码/删除/批量删除/全选全变幽灵按键。
元凶是我加改码功能那次提交。改成 `\\n`。
3. **--relay / --relay-status 崩溃**(严重)
RelayLink.start(url) 的 url 是必填参数,我把默认地址改空白时
run_agent.py 两处调用没跟着改。一处被 try 吞成「测试失败:missing
1 required positional argument」(用户以为地址填错了),
一处直接报栈。两处都补上 url。
4. 非法 deviceId(控制字符 / 路径分隔符)现在被拒(白名单:
字母数字 + - _ . :)
5. enable_relay() 第二次调用不再谎报成功(加 RelayLink.running 属性,
返回值反映真实运行状态;不再用「对象建过没有」判断)
6. snapshot() 清孤儿码后补 _save(),磁盘和内存不再脱节
== 局域网全面优化(task-2,另一个智能体完成)==
4 个真问题:
* **网段判断写死 /24** —— 在 /16 网络上 10.8.1.5 和 10.8.2.5
被判成不同网络,明明能直连却去走中转还费流量。
改成按网卡真实掩码比较。
* **广播只发一个包** —— 丢一个就整轮搜不到。改成 5 个时间点重发。
* **只探一个网段** —— 多网卡手表搜不到另一网段的电脑。改成全探。
* **走中转之后再也回不到局域网** —— 出门再回家永远挂中转。
新增定期回切(先便宜地 TCP ping,通了才动真格,
不破坏正在工作的中转连接)。
实测:局域网握手 2ms、50/50 = 100%、中位 0ms。
(跨网络那半之前拿不到数字,就是被上面那个线程崩溃挡住的 ——
现已修复。)
顺带:instance.py 单实例锁端口加 YKXMZ_LOCK_PORT 覆盖(默认不变)。
用户自己那台被控端正跑在这台机器上占着 8767/TCP + 8766/UDP,
是这次排查里好几个「测试怎么都不通」的根因。
== 服务端一键部署脚本(task-1,另一个智能体完成)==
relay/deploy.sh + relay/deploy-tune.sh,随服务端 zip 发布:
* 自动检查并安装 cloudflared
* 部署程序、装 4 个 systemd 服务、网络调优、配隧道、自检打公网地址
* **不传 --hostname 且没有 TUNNEL_TOKEN 时明确提示要填**,
不会静默用一个写死的域名(用户明确说过这个中转站是他自己用的,
不对外开放,绝不能把私人域名烧成默认值)
* 重跑 = 升级,已有数据/配对码/管理密码/凭据都不动
顺带修了发布包一直缺 site_shots/ 的老 bug
(package_remote.py 用 iterdir()+is_file() 收文件,目录进不去,
新机器装完官网截图全是 404)。
== 默认中转地址改成空白 ==
用户要求:软件默认不带任何中转地址,由用户自己填。
四处硬编码全部清空(server.py / MainActivity.java / run_agent.py /
relay.py 的注释和文档示例),全仓已无非注释的硬编码地址。
== 文档同步(task-5,另一个智能体完成)==
README / BEFORE-INSTALL / AFTER-INSTALL / 打包模板:
* 新增「软件里不带任何中转地址」告示,讲清跨网络要自己部署
* 修正 Android 版本(写的是 9.0/API28,实际 API 24 / Android 7.0)
* 删掉「地址已经内置好了」(现在是错的)
* AGENT_README 原来把作者自己的中转地址当成「内置地址」发给用户,已重写
* RELAY_README 原来写「两条隧道同时跑会互相争抢」,
和真机/CHANGELOG 矛盾;按实测改成「两种协议同时在跑(故意的)」
263 lines
9.9 KiB
Python
263 lines
9.9 KiB
Python
# -*- coding: utf-8 -*-
|
||
r'''给中转打临时插桩,抓「数据到底去哪了」。
|
||
|
||
做法:把 relay.py 复制一份到临时目录,在关键位置插入 log 调用:
|
||
* /v1/client/send -> 谁发的、多少字节、当时 to_agent 有多长
|
||
* /v1/agent/poll -> 取出多少字节、当时的 X-Relay-Session 是什么
|
||
* /v1/agent/send -> 被控端回了多少字节
|
||
* /v1/client/poll -> 手表取走多少字节
|
||
* _sync_session -> 会话令牌怎么变的
|
||
|
||
然后在同一个 repro 里跑,日志能直接指出哪一步数据消失了。
|
||
'''
|
||
from __future__ import annotations
|
||
|
||
import json
|
||
import os
|
||
import re
|
||
import shutil
|
||
import subprocess
|
||
import sys
|
||
import tempfile
|
||
import threading
|
||
import time
|
||
import urllib.error
|
||
import urllib.request
|
||
from pathlib import Path
|
||
|
||
ROOT = Path(__file__).resolve().parent.parent
|
||
WINDOWS = ROOT / "windows"
|
||
RELAY_PORT = 8126
|
||
AGENT_PORT = 8127
|
||
BASE = f"http://127.0.0.1:{RELAY_PORT}"
|
||
PAIR = "135790"
|
||
UA = {"User-Agent": "LanRemote/1.8.2"}
|
||
|
||
|
||
def instrument(src: str) -> str:
|
||
"""在 relay.py 的关键位置插 log。"""
|
||
# 1. client/send
|
||
src = src.replace(
|
||
""" data = self._body()
|
||
RELAY.client_put(session, data)
|
||
self._json(200, {"ok": True, "queued": len(data)})""",
|
||
""" data = self._body()
|
||
_dev = RELAY.devices.get(session.device_id)
|
||
log(f"[INS] client/send {len(data)}B -> to_agent 现有 "
|
||
f"{len(_dev.to_agent) if _dev else '?'}B,会话 {token[:8]},"
|
||
f"current_session={( _dev.current_session or '-')[:8] if _dev else '?'}")
|
||
RELAY.client_put(session, data)
|
||
self._json(200, {"ok": True, "queued": len(data)})""", 1)
|
||
|
||
# 2. agent/poll
|
||
src = src.replace(
|
||
""" session_at_start = device.current_session
|
||
data = RELAY.agent_take(device, POLL_SECONDS)""",
|
||
""" session_at_start = device.current_session
|
||
log(f"[INS] agent/poll 进入,to_agent={len(device.to_agent)}B,"
|
||
f"current_session={(device.current_session or '-')[:8]}")
|
||
data = RELAY.agent_take(device, POLL_SECONDS)
|
||
log(f"[INS] agent/poll 返回 {len(data)}B,"
|
||
f"current_session={(device.current_session or '-')[:8]}")""", 1)
|
||
|
||
# 3. agent/send
|
||
src = src.replace(
|
||
""" data = self._body()
|
||
RELAY.agent_put(device, data)
|
||
self._json(200, {"ok": True, "queued": len(data)})""",
|
||
""" data = self._body()
|
||
log(f"[INS] agent/send {len(data)}B,active_session="
|
||
f"{(device.current_session or '-')[:8]}")
|
||
RELAY.agent_put(device, data)
|
||
self._json(200, {"ok": True, "queued": len(data)})""", 1)
|
||
|
||
# 4. client/poll
|
||
src = src.replace(
|
||
"""from __future__ import annotations""",
|
||
"""from __future__ import annotations # instrumented""", 1)
|
||
return src
|
||
|
||
|
||
def post(path, body=b"", headers=None, timeout=20.0):
|
||
h = dict(UA)
|
||
h.update(headers or {})
|
||
req = urllib.request.Request(BASE + path, data=body, headers=h, method="POST")
|
||
try:
|
||
with urllib.request.urlopen(req, timeout=timeout) as resp:
|
||
return resp.status, resp.read()
|
||
except urllib.error.HTTPError as exc:
|
||
return exc.code, exc.read()
|
||
except Exception as exc: # noqa: BLE001
|
||
return 0, str(exc).encode()
|
||
|
||
|
||
def get(path, timeout=10.0):
|
||
req = urllib.request.Request(BASE + path, headers=UA)
|
||
try:
|
||
with urllib.request.urlopen(req, timeout=timeout) as resp:
|
||
return resp.status, resp.read()
|
||
except Exception as exc: # noqa: BLE001
|
||
return 0, str(exc).encode()
|
||
|
||
|
||
class Watch:
|
||
def __init__(self, session):
|
||
self.session = session
|
||
self.buf = b""
|
||
|
||
def send(self, obj):
|
||
payload = (json.dumps(obj, ensure_ascii=False) + "\n").encode()
|
||
st, _ = post("/v1/client/send", payload, {"X-Session": self.session})
|
||
return st == 200
|
||
|
||
def reply(self, timeout=20.0):
|
||
deadline = time.time() + timeout
|
||
while time.time() < deadline:
|
||
while b"\n" in self.buf:
|
||
line, _, self.buf = self.buf.partition(b"\n")
|
||
if not line.strip():
|
||
continue
|
||
msg = json.loads(line.decode("utf-8"))
|
||
if "event" in msg:
|
||
continue
|
||
return msg
|
||
st, body = post("/v1/client/poll", b"", {"X-Session": self.session},
|
||
timeout=max(4.0, deadline - time.time()))
|
||
if st != 200:
|
||
time.sleep(0.4)
|
||
continue
|
||
self.buf += body
|
||
return None
|
||
|
||
|
||
def main() -> int:
|
||
work = Path(tempfile.mkdtemp(prefix="ins_"))
|
||
relay_proc = agent_proc = None
|
||
real_data = WINDOWS / "data"
|
||
backup = WINDOWS / "data_bk_ins"
|
||
had = real_data.exists()
|
||
if had:
|
||
if backup.exists():
|
||
shutil.rmtree(backup, ignore_errors=True)
|
||
shutil.move(str(real_data), str(backup))
|
||
try:
|
||
# 插桩版的 relay.py
|
||
relay_src = (ROOT / "relay" / "relay.py").read_text(encoding="utf-8")
|
||
inst = instrument(relay_src)
|
||
if "[INS]" not in inst:
|
||
print(" ✗ 插桩失败:锚点没匹配上")
|
||
return 1
|
||
n = inst.count("[INS]")
|
||
print(" 插桩了 %d 处" % n)
|
||
relay_dir = work / "relaydir"
|
||
shutil.copytree(ROOT / "relay", relay_dir)
|
||
(relay_dir / "relay.py").write_text(inst, encoding="utf-8",
|
||
newline="\n")
|
||
|
||
real_data.mkdir(parents=True, exist_ok=True)
|
||
sys.path.insert(0, str(WINDOWS))
|
||
from lanremote.config import Store
|
||
store = Store(real_data)
|
||
store.set_pairing_code(PAIR)
|
||
store.set("setup_done", True)
|
||
store.set("allow_new_pairing", True)
|
||
store.set("close_pair_after_first", False)
|
||
store.set("server_name", "插桩电脑")
|
||
store.set("relay_url", f"http://127.0.0.1:{RELAY_PORT}")
|
||
store.set("port", AGENT_PORT)
|
||
|
||
rdata = work / "r"
|
||
rdata.mkdir(parents=True, exist_ok=True)
|
||
env = dict(os.environ)
|
||
env.update({"RELAY_PORT": str(RELAY_PORT), "RELAY_BIND": "127.0.0.1",
|
||
"RELAY_DATA": str(rdata), "RELAY_POLL": "5",
|
||
"PYTHONIOENCODING": "utf-8", "PYTHONUNBUFFERED": "1"})
|
||
# ⚠ 用户自己的被控端正跑在这台机器上,占着单实例锁端口 8767。
|
||
# 不换端口的话测试实例一起来就被"已有实例在运行"顶掉,
|
||
# 表现为"被控端根本没登记",而且日志是空的,极难排查。
|
||
agent_env = dict(env)
|
||
agent_env["YKXMZ_LOCK_PORT"] = "18767"
|
||
relay_log = open(work / "relay.log", "wb")
|
||
relay_proc = subprocess.Popen(
|
||
[sys.executable, "-u", str(relay_dir / "relay.py")], env=env,
|
||
cwd=str(relay_dir), stdout=relay_log, stderr=subprocess.STDOUT)
|
||
for _ in range(40):
|
||
time.sleep(0.25)
|
||
if get("/v1/health")[0] == 200:
|
||
break
|
||
print(" 插桩中转起来了")
|
||
|
||
agent_log = open(work / "agent.log", "wb")
|
||
agent_proc = subprocess.Popen(
|
||
[sys.executable, "-u", str(WINDOWS / "run_agent.py"),
|
||
"--silent", "--port", str(AGENT_PORT)],
|
||
cwd=str(WINDOWS), stdout=agent_log, stderr=subprocess.STDOUT,
|
||
env=agent_env)
|
||
|
||
device = None
|
||
for _ in range(40):
|
||
time.sleep(0.5)
|
||
st, body = get("/v1/health")
|
||
if st == 200:
|
||
snap = json.loads(body.decode())
|
||
if snap.get("devices"):
|
||
device = snap["devices"][0]
|
||
break
|
||
print(" 被控端登记: %s" % (device["name"] if device else "**没有**"))
|
||
|
||
st, body = post("/v1/client/attach", json.dumps(
|
||
{"code": PAIR, "name": "插桩手表"}).encode(),
|
||
{"Content-Type": "application/json"})
|
||
att = json.loads(body.decode())
|
||
watch = Watch(att.get("session", ""))
|
||
print(" 手表接入: ok=%s" % att.get("ok"))
|
||
|
||
steps = [{"cmd": "pair", "deviceId": "w1", "name": "手表", "code": PAIR},
|
||
{"cmd": "ping"}, {"cmd": "ping"}, {"cmd": "ping"}]
|
||
for i, msg in enumerate(steps, 1):
|
||
t0 = time.time()
|
||
watch.send(msg)
|
||
got = watch.reply(timeout=15.0)
|
||
dt = int((time.time() - t0) * 1000)
|
||
print(" %d. %-8s %s %dms" % (i, msg["cmd"],
|
||
"OK" if got else "**超时**", dt))
|
||
if got is None:
|
||
print(" 卡住了,停止")
|
||
break
|
||
time.sleep(1.0)
|
||
|
||
print()
|
||
print(" ════ 插桩日志(中转侧数据流)════")
|
||
relay_log.flush()
|
||
txt = (work / "relay.log").read_text(encoding="utf-8", errors="replace")
|
||
for line in txt.splitlines():
|
||
if "[INS]" in line:
|
||
print(" " + line[:130])
|
||
|
||
print()
|
||
print(" ════ 被控端日志 ════")
|
||
agent_log.flush()
|
||
txt = (work / "agent.log").read_text(encoding="utf-8", errors="replace")
|
||
for line in txt.splitlines()[-20:]:
|
||
print(" " + line[:120])
|
||
return 0
|
||
finally:
|
||
for p in (agent_proc, relay_proc):
|
||
if p is not None:
|
||
try:
|
||
p.terminate()
|
||
p.wait(timeout=6)
|
||
except Exception: # noqa: BLE001
|
||
try:
|
||
p.kill()
|
||
except Exception: # noqa: BLE001
|
||
pass
|
||
shutil.rmtree(work, ignore_errors=True)
|
||
shutil.rmtree(real_data, ignore_errors=True)
|
||
if had:
|
||
shutil.move(str(backup), str(real_data))
|
||
|
||
|
||
if __name__ == "__main__":
|
||
sys.exit(main())
|