 xmz7891
|
c97d7e8961
|
修「去密码硬编码」的第三批机械替换伤
上一轮的批量替换又留下四处伤，全都表现成"测试红了/文件坏了"，
根子都在替换脚本不看上下文。
== 1. import os 插到了 __future__ 前面 ==
fix_password_round2.py 给文件补 `import os` 时，直接插在第一个
import 之前。但那个文件是：
'''
from __future__ import annotations
import ast
被插成了 `import os` 在 `from __future__` 之前 ——
Python 要求 __future__ 必须是第一条语句，于是整个文件语法错：
SyntaxError: from __future__ imports must occur at the beginning
受影响：_build/test_response_headers.py
== 2. 把中文说明塞进了字符串字面量 ==
同一个脚本里：
t = t.replace(OLD, "（从 RELAY_ADMIN_PASSWORD 环境变量读）")
凡 `"@<老密码>"` 这种形状（@ 在引号里）都被替换成字符串：
PASS = "@（从 RELAY_ADMIN_PASSWORD 环境变量读）"
ADMIN_PASS = "@（从 RELAY_ADMIN_PASSWORD 环境变量读）"
本该是 `(os.environ.get("RELAY_ADMIN_PASSWORD") or "").strip()`。
受影响：_build/test_response_headers.py、
_build/split_site_admin2.py
== 3. 测试自己起的中转拿不到密码（第二批漏了一个文件）==
test_response_headers.py 也是自己起本地中转，然后用 PASS 登录。
PASS 从环境变量取，本地跑时是空的 -> 登录失败 -> 401
-> 表现成「登录拿到 cookie（没拿到）」，看起来像登录功能坏了。
和上一批同样的修法：没设环境变量就**生成一个本次运行专用的随机密码**，
同时通过 env 和 data/admin_password.txt 告诉那台测试中转。
== 4. 模块级调用写在函数定义之前 ==
test_relay_admin_login.py 里 `PASSWORD = _admin_password()`
在模块级第 29 行，而 `def _admin_password()` 在第 64 行 ——
执行时 NameError（pyflakes 也报 undefined name）。
把这个赋值挪到 helper 定义之后。
== 5. 顺手清掉一个语法就是坏的死脚本 ==
_build/poll60_patch.py 第 49 行有个未闭合字符串，从来跑不起来，
也没人引用 —— 它是一次性补丁，改完就该删。留着只会让静态检查
一直报错。已 git rm（历史里还能查到内容）。
== 6. verify_all.ps1 的 Show-Fails 会崩 ==
$lines | Select-Object -Last 1 | ForEach-Object {
Write-Host " $($_.Trim())"
}
子进程的 stderr 被塞进输出流时，$_ 是 ErrorRecord，没有 .Trim()
方法 -> 抛 "does not contain a method named 'Trim'"，
反而把真正的失败信息盖住了。改成先转字符串再 Trim。
== 验证 ==
网页响应头 12/12 通过
管理页设备控制 23/23 通过
管理页批删 24/24 通过
管理页按钮 10/10 通过
管理页网络体检 15/15 通过
密码随机生成 11/11 通过
全仓语法检查 0 个问题
教训再记一笔：**机械替换必须替换完就验证**，
而且兜底的 str.replace 一定要慎用 —— 它不看上下文，
"函数调用进引号" 和 "中文说明进字符串" 都是这么来的。
|
2026-10-07 21:04:59 +08:00 |
|
|
xmz7891
|
430923dd84
|
🔐 修一个严重安全问题：管理密码被硬编码进公开仓库
== 问题 ==
relay/relay.py 第 138 行：
ADMIN_PASSWORD_DEFAULT = "@REVAedu120830"
而且 _tests/ 和 _build/ 下 11 个文件里也写着明文密码。
这个仓库是要推到**公开的 Gitee / EzHub** 的，等于：
git 仓库 = 公开的
管理页路径 /zhongzhuanzhan = 公开的（就写在代码里）
公网地址 = 公开的
密码 = 也在仓库里
**= 把管理台的门钥匙挂在门上。**
而且 git 历史删不掉 —— 只是在最新提交里删掉没用，
读过仓库的人已经从历史里拿到了。
== 修法 ==
### 1. 代码里不再有任何固定密码
relay.py 的 admin_password() 改成和 admin_key 一样的做法：
环境变量 RELAY_ADMIN_PASSWORD
-> data/admin_password.txt（600 权限，不进仓库）
-> 都没有就**首次启动随机生成**，写进那个文件并打印一次
这样全新部署的密码天然是随机的、每台不同。
### 2. 11 个文件里的明文密码清掉
_tests/ acceptance.py, test_relay_admin_login.py,
test_admin_batch_delete.py, test_admin_button.py,
test_admin_device_control.py, test_admin_js.py,
test_admin_netcheck.py
_build/ check_web.sh, verify_site_admin.sh,
split_site_admin2.py, test_response_headers.py
统一改成从 RELAY_ADMIN_PASSWORD 环境变量读，
拿不到就从部署的 data/admin_password.txt 读，
再拿不到就**跳过需要登录的用例**（而不是编个假密码硬试）。
### 3. 真机上的密码已经轮换
设备上的 data/admin_password.txt 已换成用户指定的新密码，
并实测：
✓ 新密码能登录（HTTP 302 + lr_admin cookie）
✓ 老密码登不上了
✓ 新密码无换行（printf 不带 \n，免得读出来多一个字符）
顺带：重启中断了 relay-stability 的 A/B 对照 A 组（69 个样本），
分析数据时会标注这一段。
== 新增测试 ==
_tests/test_admin_password_gen.py 验证六件事：
1. 全新部署会生成随机密码（不是那个老密码）
2. 密码长度 >= 12、纯字母数字
3. 用生成的密码能登录管理页
4. **老密码登不进去**
5. 重启后密码不变（从文件读回来）
6. **两次全新部署的密码不一样**（真的是随机的）
== 还没做的（等用户决定）==
git **历史**里还有那个密码。要彻底清掉得重写历史 + 强制推送，
这是个危险操作，按用户的规矩必须他确认之后才做。
而且**光删历史不够** —— 密码已经公开过，必须换（已换）。
|
2026-10-07 20:49:33 +08:00 |
|