mirror of
https://gitee.com/xmz7891/ykxmz.git
synced 2026-10-18 14:50:45 +00:00
xmz7891
9bdd36cf56
用户报了三件事，一起处理。
== 一、发行版介绍里泄漏了服务器公网 IP ==
_build/CHANGELOG.md 的 2.1.3 那节写了服务器公网 IP，而发布脚本
把这个文件当"更新说明"传给 Gitee，于是发行版介绍里就带出去了。
为什么 IP 不该公开：直接打 IP 就绕过 Cloudflare 了 ——
没有 CDN、没有防护、没有速率限制。源站本来就不该被人知道是谁
（这也是为什么 nginx 那层只认域名、中转只绑 127.0.0.1）。
处理：
* 已发的 v2.1.3 发行版介绍用 API 改掉（扫了全部 11 个发行版，只它一个）
* 仓库里 16 个文件里的 IP 全部换成域名或占位符
* 运维脚本改成从环境变量读，公开仓库里不留具体地址
顺手发现**更严重的一处**：旧管理员密码硬编码在
docs/AFTER-INSTALL.txt（会打进服务端 zip）和
installer/license-combined.txt（**安装包的许可页，每个装的人都能看到**）里。
而用户当前密码就含这个串，等于公开了一半。一并清掉，
公开材料里改成"密码在你服务器上"的说明。
== 二、手表端填了地址却连不上 ==
用户截图：填域名（没写协议头）-> 点「检查服务器」报
MalformedURLException: no protocol: <域名>/v1/health
根因：补协议头的逻辑只写在连接那条路（RelayTransport 的构造函数）上，
checkServer() 直接拿原始字符串拼 URL —— 同一个地址，连接能用、检查却崩。
修法：把规范化收到 relayUrl() 这**唯一一个入口**，所有调用方都从那里取。
规则和电脑端一致：IP / localhost / *.local -> http，域名 -> https。
顺带补了 _tests/test_relay_url_normalize.py 盯住它（含 9 个用例实算）。
== 三、Gitee 的 README 被判"可能含有违规信息" ==
Gitee 挂的是软性公告（可自行修改或申诉），但确实挂在那。
逐项看了 README，改掉几处**不准确**且容易误判的表述：
* 原文写"看看学生机是不是在打游戏" —— 这是**错的**：
被控端只装在一台电脑上（讲台那台），根本看不到别的机器。
既不准确又像监控软件，改成"作用于被控端那一台"
* 课堂那几项功能如实说明：改的是本机 hosts、结束的是本机进程，
都可逆，下课点「一键恢复全部」就还原
* "免杀对抗""伪装成系统进程"这类词（虽然是在说"我们不做这些"）
本身就在审核词库里，换成正面的说法
* 成品文件名从 1.8.2 更新到当前版本，并且以后发版自动跟着改
（把 README 加进了 bump 脚本；以前一直停在 1.8.2，
用户照着 README 去 releases 根本找不到那个文件）
== 四、做这件事时自己踩的坑（都写进注释了）==
* 批量替换时**没排除 gitignore 的文件**，把本地密码文件也改了
* 替换时往字典字面量中间塞注释，**弄坏了 5 个文件的语法**
* 修的时候手写"坏文件列表"，漏了 2 个 ——
改成全仓库扫模式（列表是观察到的，不是穷尽的）
* 清理工具自己又成了泄漏点（脚本里写了要匹配的 IP/密码），
加了 _build/precommit_leak_gate.py 当闸门才抓到
教训：批量替换要先排除不该碰的文件，而且不能往表达式中间塞东西。
== 按用户要求：测完再提交 ==
全量回归 41 个阶段全 PASS（2 项因锁屏 SKIP）
601 个 Python 文件语法通过
pyflakes 干净
安装包完整周期 20/20
对外材料清洁度 15/15
提交前泄漏闸门通过
（这条信息是补写的：上一个提交因为 heredoc 把提交说明打到屏幕而不是
文件，信息变成空的 "remove temp" 了，内容本身是对的。）
131 lines
4.1 KiB
Python
131 lines
4.1 KiB
Python
# -*- coding: utf-8 -*-
|
||
r'''修我自己 scrub 时弄坏的 3 个文件。
|
||
|
||
== 怎么坏的 ==
|
||
|
||
scrub_old_password.py 里对这个模式：
|
||
|
||
{"password": "<六位数字>"}
|
||
|
||
做了个"聪明"的替换：
|
||
|
||
{"password": "" # 见下方注释：不再写死}
|
||
|
||
—— 把 `"<六位数字>"` 换成了带注释的东西，**塞进了字典字面量中间**，
|
||
于是 `{` 永远不闭合，三个文件全语法错。
|
||
|
||
教训：**批量替换不能往表达式中间塞注释**。
|
||
这次改成把整行替换掉，并在文件顶部加一个读密码的辅助函数。
|
||
|
||
（同一批被改的其它文件只是字符串内容变了，语法没坏，
|
||
这里只修真正语法出错的三个。）
|
||
|
||
== 顺带 ==
|
||
|
||
这个脚本本身也演示了"怎么在运维脚本里拿密码而不写死"：
|
||
从环境变量或 _build/.admin_password 读（后者已 gitignore）。
|
||
'''
|
||
from __future__ import annotations
|
||
|
||
import re
|
||
import sys
|
||
from pathlib import Path
|
||
|
||
ROOT = Path(__file__).resolve().parent.parent
|
||
|
||
BROKEN_LINE = re.compile(
|
||
r'^(\s*)form = urllib\.parse\.urlencode\(\{"password": ""'
|
||
r'\s*# 见下方注释：不再写死\}\)\.encode\(\)\s*$')
|
||
|
||
# ⚠ 不要手写文件列表。
|
||
#
|
||
# 第一版我列了三个文件名（照语法检查的输出抄的），
|
||
# 结果一跑发现还有两个同样的坏文件没修 —— 列表是**观察到的**，
|
||
# 不是**穷尽的**。改成全仓库扫这个模式，漏不掉。
|
||
SCAN_DIRS = ("_build", "_tests", "windows", "relay", "installer")
|
||
|
||
REPLACEMENT = (
|
||
'{indent}form = urllib.parse.urlencode('
|
||
'{{"password": _admin_password()}}).encode()')
|
||
|
||
HELPER = '''
|
||
|
||
# ⚠ 不再把密码写死在脚本里。
|
||
#
|
||
# 这个仓库是公开的，密码写进来就等于公开（2026-10-08 踩过：
|
||
# 旧密码曾被写进安装包的许可页，每个装软件的人都能看到）。
|
||
# 现在从环境变量或 _build/.admin_password（已 gitignore）读 ——
|
||
# 读不到就直接报错，别偷偷用一个默认值。
|
||
def _admin_password() -> str:
|
||
import os
|
||
env = os.environ.get("YKXMZ_ADMIN_PASSWORD", "").strip()
|
||
if env:
|
||
return env
|
||
f = Path(__file__).resolve().parent / ".admin_password"
|
||
if f.is_file():
|
||
return f.read_text(encoding="utf-8").strip()
|
||
raise SystemExit(
|
||
"没拿到管理密码：设 YKXMZ_ADMIN_PASSWORD，"
|
||
"或把密码写进 _build/.admin_password")
|
||
'''
|
||
|
||
|
||
def main() -> int:
|
||
print("=" * 58)
|
||
print(" 修被 scrub 弄坏的 3 个文件")
|
||
print("=" * 58)
|
||
|
||
fixed = 0
|
||
# 全仓库找坏模式，而不是照抄一份文件名列表
|
||
targets = []
|
||
for d in SCAN_DIRS:
|
||
for p in sorted((ROOT / d).rglob("*.py")):
|
||
try:
|
||
if BROKEN_LINE.search(p.read_text(encoding="utf-8-sig")):
|
||
targets.append(p)
|
||
except (OSError, UnicodeDecodeError):
|
||
continue
|
||
|
||
print(" 扫到 %d 个待修文件" % len(targets))
|
||
print()
|
||
for path in targets:
|
||
rel = path.relative_to(ROOT).as_posix()
|
||
raw = path.read_bytes()
|
||
bom = raw.startswith(b"\xef\xbb\xbf")
|
||
text = raw.decode("utf-8-sig")
|
||
lines = text.splitlines()
|
||
hit = False
|
||
for i, line in enumerate(lines):
|
||
m = BROKEN_LINE.match(line)
|
||
if m:
|
||
lines[i] = REPLACEMENT.format(indent=m.group(1))
|
||
hit = True
|
||
|
||
if not hit:
|
||
continue
|
||
|
||
text = "\n".join(lines) + "\n"
|
||
|
||
# 顶部加辅助函数（放在 `import` 全部结束之后）
|
||
if "_admin_password" not in text.split("def _admin_password")[0]:
|
||
m2 = None
|
||
for m in re.finditer(r"^(?:import|from)\s+\S+.*$", text, re.M):
|
||
m2 = m
|
||
if m2:
|
||
text = text[:m2.end()] + "\n" + HELPER + text[m2.end():]
|
||
else:
|
||
text = HELPER + text
|
||
|
||
path.write_bytes((b"\xef\xbb\xbf" if bom else b"")
|
||
+ text.encode("utf-8"))
|
||
print(" ✓ %s" % rel)
|
||
fixed += 1
|
||
|
||
print()
|
||
print(" 修了 %d 个" % fixed)
|
||
return 0
|
||
|
||
|
||
if __name__ == "__main__":
|
||
sys.exit(main())
|