mirror of
https://gitee.com/xmz7891/ykxmz.git
synced 2026-10-18 21:50:44 +00:00
用户报了三件事,一起处理。
== 一、发行版介绍里泄漏了服务器公网 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" 了,内容本身是对的。)
120 lines
4.0 KiB
Python
120 lines
4.0 KiB
Python
# -*- coding: utf-8 -*-
|
|
r'''清掉 _build/ 里剩下的 5 处明文管理密码。
|
|
|
|
形状:
|
|
_build/check_web.sh:29 -d 'username=xmz7891&password=<旧>' "$BASE/admin/login"
|
|
_build/verify_site_admin.sh:38 同上
|
|
_build/split_site_admin2.py:10 注释里
|
|
_build/split_site_admin2.py:22 ADMIN_PASS = "<旧>"
|
|
_build/test_response_headers.py:28 PASS = "<旧>"
|
|
|
|
改法:
|
|
* shell 脚本改成从环境变量读:${RELAY_ADMIN_PASSWORD:?请先设置}
|
|
* Python 改成 os.environ.get("RELAY_ADMIN_PASSWORD")
|
|
* 注释里把密码删掉,改成说明怎么取
|
|
'''
|
|
from __future__ import annotations
|
|
|
|
import re
|
|
import sys
|
|
from pathlib import Path
|
|
|
|
ROOT = Path(__file__).resolve().parent.parent
|
|
OLD = "REVA" + "edu<改从环境变量读>"
|
|
|
|
passed = 0
|
|
failed = 0
|
|
|
|
|
|
def report(name, ok, detail=""):
|
|
global passed, failed
|
|
if ok:
|
|
passed += 1
|
|
print(" ✓ " + name)
|
|
else:
|
|
failed += 1
|
|
print(" ✗ " + name + ((" —— " + str(detail)) if detail else ""))
|
|
|
|
|
|
def main() -> int:
|
|
# ---- shell 脚本:改成读环境变量 ----
|
|
for name in ("check_web.sh", "verify_site_admin.sh"):
|
|
p = ROOT / "_build" / name
|
|
if not p.is_file():
|
|
report(name, False, "不存在")
|
|
continue
|
|
t = p.read_text(encoding="utf-8")
|
|
if OLD not in t:
|
|
report(name, True, "已干净,跳过")
|
|
continue
|
|
before = t
|
|
t = re.sub(
|
|
r"-d\s+'username=[^']*&password=[^']*'",
|
|
'-d "username=xmz7891&password=$RELAY_ADMIN_PASSWORD"',
|
|
t)
|
|
if t == before:
|
|
report(name, False, "替换没生效")
|
|
continue
|
|
if OLD in t:
|
|
report(name, False, "还有残留")
|
|
continue
|
|
# 在文件顶部加一句检查,免得忘设环境变量时报错莫名其妙
|
|
lines = t.splitlines()
|
|
insert_at = 0
|
|
for i, line in enumerate(lines[:12]):
|
|
if line.startswith("#!"):
|
|
insert_at = i + 1
|
|
break
|
|
lines.insert(insert_at, "")
|
|
lines.insert(insert_at + 1,
|
|
'# ⚠ 管理密码不再写在这里,从环境变量取:')
|
|
lines.insert(insert_at + 2,
|
|
'# export RELAY_ADMIN_PASSWORD=...')
|
|
lines.insert(insert_at + 3,
|
|
': "${RELAY_ADMIN_PASSWORD:?请先 export RELAY_ADMIN_PASSWORD}"')
|
|
p.write_text("\n".join(lines) + "\n", encoding="utf-8", newline="\n")
|
|
report(name, True, "改成读环境变量")
|
|
|
|
# ---- Python:改成 os.environ ----
|
|
for name in ("split_site_admin2.py", "test_response_headers.py"):
|
|
p = ROOT / "_build" / name
|
|
if not p.is_file():
|
|
report(name, False, "不存在")
|
|
continue
|
|
t = p.read_text(encoding="utf-8")
|
|
if OLD not in t:
|
|
report(name, True, "已干净,跳过")
|
|
continue
|
|
before = t
|
|
# ADMIN_PASS = "..." / PASS = "..."
|
|
t = re.sub(r'(\w*PASS\w*)\s*=\s*"%s"' % re.escape(OLD),
|
|
r'\1 = (os.environ.get("RELAY_ADMIN_PASSWORD") or "").strip()',
|
|
t)
|
|
# 注释里的
|
|
t = t.replace(OLD, "(从 RELAY_ADMIN_PASSWORD 环境变量读)")
|
|
t = t.replace('"%s"' % OLD, '""')
|
|
t = t.replace(OLD, "")
|
|
if OLD in t:
|
|
report(name, False, "还有残留")
|
|
continue
|
|
if t == before:
|
|
report(name, False, "替换没生效")
|
|
continue
|
|
# 确保有 import os
|
|
if "import os" not in t:
|
|
m = re.search(r"^(import |from )", t, re.M)
|
|
if m:
|
|
t = t[:m.start()] + "import os\n" + t[m.start():]
|
|
else:
|
|
t = "import os\n" + t
|
|
p.write_text(t, encoding="utf-8", newline="\n")
|
|
report(name, True, "改成读环境变量")
|
|
|
|
print()
|
|
print(" 改了 %d 个,失败 %d 个" % (passed, failed))
|
|
return 0 if failed == 0 else 1
|
|
|
|
|
|
if __name__ == "__main__":
|
|
sys.exit(main())
|