mirror of
https://gitee.com/xmz7891/ykxmz.git
synced 2026-10-19 15:20:44 +00:00
修用户报的三个问题 + 揪出我自己检查工具的盲区
用户报:
1. 官网 502
2. 局域网连接:输入配对码提示连上了但没反应,得推出去点设备才能连
3. 被控端安装时要能设中转服务器;之后双击 exe 也该能改配对码 /
开关新设备 / 中转地址 —— 但现在双击没反应
────────────────────────────────────────
问题 1:502(暂态,已恢复)
────────────────────────────────────────
你截图是 21:24:56,隧道日志显示 21:22:51 和 21:26:44 各重启了一次
—— 502 就发生在那个空窗里。现在三个服务都 active、10 条连接、
本机 200(3ms)、公网 200(2.6s)。
(重启是谁触发的还没查清,watchdog 的嫌疑最大,下一轮跟进。)
────────────────────────────────────────
问题 2:局域网连上后没反应(真 bug,已修)
────────────────────────────────────────
手表上有**四个**面板,靠手工 setVisibility 切换:
connectPanel 首页
controlPanel 控制面板
mirrorPanel 投屏
lanPanel 局域网连接页 <- 后加的
而 showControlPanel() 只藏了前两个:
connectPanel.setVisibility(GONE);
controlPanel.setVisibility(VISIBLE);
if (mirrorPanel != null) mirrorPanel.setVisibility(GONE);
// lanPanel 没藏!
于是从「局域网连接」页填完码、连接成功之后:
connect() 成功 -> showControlPanel() -> 控制面板显示了
但 lanPanel 还盖在最上层 -> 用户看到的还是局域网页面
表现就是用户说的「提示连上了,但没反应,得推出去到设备列表
点击设备才能连接」—— 推出去把 lanPanel 收掉,才看到底下。
**根子在于"每处切换都手写一串 setVisibility"**,加第四个面板时
必然漏。改成统一入口:
private void showOnly(View keep) // 藏掉所有面板,只留一个
6 个切换点全部走它。以后加面板只要在 showOnly 里加一行。
────────────────────────────────────────
问题 3:双击 exe 没反应(功能缺失,已补)
────────────────────────────────────────
原来是两件事叠在一起:
**3a. 装完之后双击永远静默**(main() 最后几行)
if not store.get("setup_done", False):
return run_setup(store)
return run_silent(store, None) # <- 什么都不弹
改成:没参数 + 已设置过 -> **打开设置窗口**。
后台运行走显式 --silent(开机自启和安装包都用它,不受影响)。
**3b. 设置窗口里没有中转服务器**
SetupWindow 原来只有 配对码 / 开机自启 / 放行防火墙。
现在加了:
* 中转服务器地址(可不填;只在一个 WiFi 里用就不需要)
* 允许新设备接入(开关,只在"设置"模式给)
并且给 SetupWindow 加了 mode:
"first" 第一次设置(防火墙 + 自启 + 转静默,老行为不变)
"settings" 已装好,只改配置
**3c. 顺带修了一个会静默失败的东西**
改完发现打包出来的 exe 一跑就 ModuleNotFoundError:
No module named 'lanremote.ui' —— **PyInstaller 的 --hidden-import
里没有 ui**。它是 tkinter 那套设置窗口的实现,而静默那条路
不需要它,静态分析就找不到。源码跑得好好的,只有 exe 会崩。
已加 --hidden-import "lanremote.ui",并写了
_tests/test_double_click_window.py 针对**打包好的 exe** 验:
✅ 进程没崩 / 窗口弹出来了 / 标题是「ykxmz · 设置」
────────────────────────────────────────
额外揪出来的:我的检查工具一直在骗我
────────────────────────────────────────
反复被同一个问题咬了三次:补丁脚本为了加 import os / import re,
把它插到了 `from __future__` 前面,整个文件语法错。
而**我前面几次说的「全仓语法检查 0 个问题」是假绿**:
ast.parse(src) -> 通过(只解析,不做编译期检查)
compile(src, ...) -> SyntaxError: from __future__ imports
must occur at the beginning of the file
`__future__` 的位置是**编译期**检查,ast.parse 根本不管。
所以改成用 compile(),并写成
_tests/check_python_syntax.py(回归阶段 3a7),
它还会专门把"ast.parse 会放过、compile 会拦"的文件标出来。
实测:全仓 533 个文件,用 compile() 扫出 1 个坏的(ui.py),
而它正是让"双击没反应"的最后一个原因。
────────────────────────────────────────
还有一个重要发现
────────────────────────────────────────
**你一直在跑的是 10-06 的旧 exe**:
C:\Users\shu\Desktop\ykxmz\ykxmz.exe 10-06 17:43 <- 你双击的这个
LanRemote\dist_exe\ykxmz\ykxmz.exe 10-07 21:5x <- 修复都在这
它的 silent.log 正在反复报:
[21:15:59] 连接断了,重开一次再试(The read operation timed out)
[21:16:42] 中转连接出问题:RelayError: 请求失败: ...
[21:18:34] 已登记到中转:配对码 111111
这就是我修的那个 126 秒长轮询问题 —— 旧版超时太短反复掉线。
新版把客户端超时改成 180 秒了。
**所以这三个修复要装上新版才生效。**
This commit is contained in:
+86
-6
@@ -22,6 +22,7 @@
|
||||
"""
|
||||
from __future__ import annotations
|
||||
|
||||
import re
|
||||
import tkinter as tk
|
||||
from pathlib import Path
|
||||
|
||||
@@ -111,8 +112,14 @@ class SetupWindow:
|
||||
|
||||
CODE_LEN = 6
|
||||
|
||||
def __init__(self, store, suggested: str = ""):
|
||||
def __init__(self, store, suggested: str = "",
|
||||
mode: str = "first"):
|
||||
"""mode:
|
||||
"first" 第一次设置(走防火墙 + 自启 + 转静默)
|
||||
"settings" 已装好,只改配置(配对码/中转/接入开关)
|
||||
"""
|
||||
self.store = store
|
||||
self.mode = mode if mode in ("first", "settings") else "first"
|
||||
self.suggested = (suggested or store.pairing_code()
|
||||
or store.new_pairing_code())
|
||||
self.result: dict | None = None
|
||||
@@ -128,7 +135,8 @@ class SetupWindow:
|
||||
on_ready 是窗口建好之后的回调(自动化测试用)。
|
||||
"""
|
||||
self.root = tk.Tk()
|
||||
self.root.title("ykxmz · 第一次设置")
|
||||
self.root.title("ykxmz · " + (
|
||||
"第一次设置" if self.mode == "first" else "设置"))
|
||||
self.root.configure(bg=BG)
|
||||
self.root.resizable(False, False)
|
||||
self._set_icon()
|
||||
@@ -234,7 +242,10 @@ class SetupWindow:
|
||||
titles.pack(side="left", anchor="w")
|
||||
tk.Label(titles, text="ykxmz", bg=BG, fg=FG, font=FONT_TITLE).pack(
|
||||
anchor="w")
|
||||
tk.Label(titles, text="远程控制 · 第一次设置", bg=BG, fg=DIM,
|
||||
tk.Label(titles,
|
||||
text=("远程控制 · 第一次设置" if self.mode == "first"
|
||||
else "远程控制 · 设置"),
|
||||
bg=BG, fg=DIM,
|
||||
font=FONT_SUB).pack(anchor="w")
|
||||
|
||||
# ================= 配对码卡片 =================
|
||||
@@ -279,6 +290,41 @@ class SetupWindow:
|
||||
justify="left", anchor="w")
|
||||
self.status.pack(fill="x", pady=(6, 0))
|
||||
|
||||
# ================= 中转服务器卡片 =================
|
||||
# 用户要求:安装时要能设中转服务器;装完之后双击 exe 也能回来改。
|
||||
#
|
||||
# 为什么不填也能用:手表和电脑在同一个 WiFi 时走局域网直连,
|
||||
# 根本不需要中转。只有"在办公室管教室"这种跨网络场景才要填。
|
||||
relay_card = self._card(page)
|
||||
relay_card.pack(fill="x", pady=(12, 0))
|
||||
relay_inner = tk.Frame(relay_card.winfo_children()[0], bg=CARD)
|
||||
relay_inner.pack(fill="x", padx=16, pady=14)
|
||||
|
||||
tk.Label(relay_inner, text="中转服务器(可以不填)", bg=CARD, fg=FG,
|
||||
font=FONT_LABEL).pack(anchor="w")
|
||||
|
||||
self.relay_entry = tk.Entry(
|
||||
relay_inner, bg="#f6f8fa", fg=FG, relief="flat",
|
||||
font=FONT_SUB, insertbackground=ACCENT)
|
||||
self.relay_entry.pack(fill="x", pady=(8, 0), ipady=6)
|
||||
_saved_relay = str(self.store.get("relay_url", "") or "").strip()
|
||||
if _saved_relay:
|
||||
self.relay_entry.insert(0, _saved_relay)
|
||||
|
||||
tk.Label(relay_inner,
|
||||
text="手表和电脑在同一个 WiFi 时不需要填 —— 会直接走"
|
||||
"局域网,几十毫秒。\n"
|
||||
"要在别的地方(比如办公室)管这台电脑时才填,"
|
||||
"填你自己那台中转服务器的地址。",
|
||||
bg=CARD, fg=MUTED, font=FONT_SMALL, justify="left",
|
||||
wraplength=396).pack(anchor="w", pady=(8, 0))
|
||||
|
||||
self.relay_status = tk.Label(relay_inner, text="", bg=CARD,
|
||||
fg=DANGER, font=FONT_SMALL,
|
||||
wraplength=396, justify="left",
|
||||
anchor="w")
|
||||
self.relay_status.pack(fill="x", pady=(6, 0))
|
||||
|
||||
# ================= 选项卡片 =================
|
||||
options = self._card(page)
|
||||
options.pack(fill="x", pady=(12, 0))
|
||||
@@ -291,9 +337,22 @@ class SetupWindow:
|
||||
self.var_auto = tk.BooleanVar(value=True)
|
||||
self._check(opt_inner, "开机自动运行", self.var_auto,
|
||||
"开机后在后台跑,不用手动开")
|
||||
self.var_firewall = tk.BooleanVar(value=True)
|
||||
self._check(opt_inner, "放行防火墙", self.var_firewall,
|
||||
"会弹一次管理员确认,然后黑窗口一闪而过")
|
||||
# 「允许新设备接入」只在设置模式给 ——
|
||||
# 第一次设置时当然是允许的(不然手表连不上)。
|
||||
if self.mode == "settings":
|
||||
self.var_allow_new = tk.BooleanVar(
|
||||
value=bool(self.store.get("allow_new_pairing", True)))
|
||||
self._check(opt_inner, "允许新设备接入", self.var_allow_new,
|
||||
"关掉之后,只有已经配对过的设备能连上;"
|
||||
"新设备会被拒绝")
|
||||
else:
|
||||
self.var_allow_new = None
|
||||
|
||||
self.var_firewall = tk.BooleanVar(
|
||||
value=(self.mode == "first"))
|
||||
if self.mode == "first":
|
||||
self._check(opt_inner, "放行防火墙", self.var_firewall,
|
||||
"会弹一次管理员确认,然后黑窗口一闪而过")
|
||||
tk.Label(opt_inner,
|
||||
text="没放行成功的话,配对码不会提交给服务器,"
|
||||
"会告诉你怎么处理。",
|
||||
@@ -389,16 +448,37 @@ class SetupWindow:
|
||||
DANGER)
|
||||
return
|
||||
|
||||
# 中转地址:允许空(空 = 只走局域网)。
|
||||
# 填了的话做一次最基本的格式检查,免得用户把域名打错还找不到原因。
|
||||
relay = self.relay_entry.get().strip()
|
||||
if relay:
|
||||
if not relay.startswith(("http://", "https://")):
|
||||
relay = "https://" + relay
|
||||
# 只留 协议://主机[:端口],去掉多余路径
|
||||
_m = re.match(r"^(https?://[^/\s]+)", relay)
|
||||
if not _m:
|
||||
self._set_relay_status("中转地址看着不对,检查一下", DANGER)
|
||||
return
|
||||
relay = _m.group(1).rstrip("/")
|
||||
self.result = {
|
||||
"code": cleaned,
|
||||
"autostart": bool(self.var_auto.get()),
|
||||
"firewall": bool(self.var_firewall.get()),
|
||||
"relay": relay,
|
||||
"allow_new": (None if self.var_allow_new is None
|
||||
else bool(self.var_allow_new.get())),
|
||||
}
|
||||
try:
|
||||
self.root.quit()
|
||||
except tk.TclError:
|
||||
pass
|
||||
|
||||
def _set_relay_status(self, text: str, color: str) -> None:
|
||||
try:
|
||||
self.relay_status.configure(text=text, fg=color)
|
||||
except tk.TclError:
|
||||
pass
|
||||
|
||||
def _cancel(self) -> None:
|
||||
self.result = None
|
||||
try:
|
||||
|
||||
Reference in New Issue
Block a user