 liuzhuohua
|
d0a72bbb5f
|
fix(release): Release 附件版本号单一真源 + 堵住「注释掉整行代码」型注入残留
发 v1.0.9 前发现两个真问题，都会直接影响用户拿到的附件。
一、docker-compose.yml 里的镜像标签是硬编码的 1.0.8
这两个文件是**直接挂到 GitHub Release 上的附件**，用户照着抄命令。
早先它们硬编码具体版本号，每次发版要人工 sed 一遍 —— 这次就漏了：
附件里写着 `image: drouter:1.0.8`，而镜像 tar 加载后的标签是 `drouter:1.0.9`，
`docker compose up` 直接报「找不到镜像」。DOCKER-GUIDE.md 另有 14 处同样问题。
改成 __VERSION__ 模板，由 build-docker.sh 渲染（版本从 packaging/VERSION 读）。
渲染分两步，顺序不能反：先删 TEMPLATE-NOTE 标记块（那是给维护者看的
「这是模板」提示，留着用户会看到「1.0.9 是占位符」这种自相矛盾的话），
再替换占位符。先替换的话，标记块里的 __VERSION__ 也会被换掉。
retag-docker-image.py 同样问题：NEW_TAGS 硬编码。改完参数化。
这个 bug 更隐蔽 —— 拿 1.0.9 的镜像跑重标记会打上 1.0.8 的标签，
用户 load 完看到的标签与附件说明对不上。
t-packaging.py 补 12 条判据卡住这些（4 条已做反向验证：注入即红、还原即绿）。
判据只抓「紧挨 drouter 标识或产物文件名」的版本号 —— 一刀切拦所有 x.y 会
误伤「1.0.1 起 Web 终端绕开 init」这种历史沿革说明和「GPL-3.0」许可证名。
二、t-worktree-clean.py 抓不到「注释掉整行代码」型注入
跑回归时 t-107 红了 1 项，查出来是 t-107-inject-wg.py 把
`'env': _vpn_env(),` 注释掉了没还原（SIGTERM 打断，finally 没执行）——
和 v1.0.8 那次 `/nonexistent` 残留一模一样的坑，但**形态不同**：
不含任何哨兵字样，原判据的 4 个哨兵全都抓不到。
后果很实：_vpn_status 不返回 env，前端拿不到四态诊断，
VPN 页面的三个修复按钮全废。这正是 v1.0.8 声称修好的那个 bug。
补两条判据：
- 扫「被注释掉的整行代码」：去掉 # 后能 ast.parse 成功，且含调用/赋值/return/dict 项
- 扫 CRLF：Windows 工具写文件会整篇改行尾，sync.sh 预检才拦太晚
判据本身踩了三个坑，都记在注释里：
- dict 项 `'env': _vpn_env(),` 单独 ast.parse 会 SyntaxError（没大括号），
真正的注入反而被放过 → 补大括号再解析
- dict 项的键**通常带引号**，正则只写 ^[A-Za-z_] 认不出来 → 改成同时认引号键
- 分隔线注释 `# ------ radvd` 里的 ----- 会被解析成负号表达式 → 按前缀排掉
双向验证：4 种注入形态（赋值/dict项/bool/列表）全部精确报到行号，
基线零误报（排掉了分隔线注释和「# module: (service, action, 中文名)」这类
长得像 dict 项的文档注释）。
三、_dev/dl-verify.py（新增）
从目标机往 Windows 拉大产物时发现的坑，记成脚本省得下次重踩：
- `ssh host 'cat f' > out` 在 Windows 上做**文本模式换行转换**，二进制被破坏
（实测 deb 少 4 字节，sha256 直接对不上）→ 走 base64 再本地解码
- Windows Python 看不到 Git Bash 的 /tmp，必须给 C:/ 绝对路径
- scp 传完一定要判 sha256 —— scp/ssh 都可能「返回 0 却落地 0 字节」
- 落地走 .part + os.replace，避免写一半失败留下半个产物
顺带修了 build-docker.sh 里一处路径笔误
（packaging/docker/packaging/docker/DOCKER-GUIDE.md）和步骤编号 1/3→1/4。
|
2026-10-04 14:05:33 +08:00 |
|
|
liuzhuohua
|
ed5eb4428b
|
1.0.9 静态资源传输层优化 + 页脚版本号 + logo 回概述
## 性能：瓶颈在传输，不在代码
先量后改。helper 冷启动 0.44s（编译 22173 行）看着像瓶颈，但
drouter-helpd 是常驻守护，请求走 Unix socket，正常路径根本不 fork；
实测各 API 已是毫秒级。app.js 语法编译 18ms、顶层执行 18ms
（393 个顶层函数 / 13401 行）—— JS 执行不是瓶颈。
真瓶颈是 app.js 690,482 字节每次重传。**决定不拆包**：JS 执行只 18ms，
拆包引入首屏 waterfall 的风险大于收益。改走三条：
1. 缓存策略按「有没有指纹」区分（不是按扩展名）
- 带 ?v=<mtime> → public, max-age=31536000, immutable
- 没指纹 → 仍 no-cache（否则部署后用户拿到旧文件）
- index.html → 永远 no-store（每次都要重新注入）
2. 编码协商 br > gzip > 原文，**逐级降级**
3. 压缩结果按 (path, mtime, size, enc) 缓存
真机实测：首次 690,482 → 200,137 字节，第二次打开 0 字节（304）。
## 逐级降级不是可选项
目标机**没装 python3-brotli**（只有 C 库 libbrotli1）。最初写的
_brotli_cached 在模块缺失时 `return data, None`，而 None 是 encoding，
调用方理解为「压不动，原样输出」—— 于是 583KB 裸传，**比优化前更慢**。
改成 (None, None) 明确表示「这条路不通」，由 _encode_body 继续试 gzip。
同时修：Accept-Encoding 探测必须认 q=0（br;q=0 是「明确不要」，
子串匹配会正好发给解不开的客户端 → 满屏乱码）；编码必须进缓存键
（共用键会互相覆盖）；brotli 固定 q5（q11 只省 10KB 却要 1.5-3s）。
## 宽 except 把 bug 装成了特性
兜底只接 ImportError 这类**真·预期**的缺失，不接 TypeError/ValueError
（「str 传给要 bytes 的接口」也是 TypeError，接了等于没接）。
早先的 `except Exception` 吞掉了一个 NameError（漏注入 _comp_lock），
表现是压缩功能完好但从不生效，页面照常能开、只看响应头的判据完全发现不了。
## 版本号：建立单一真源 packaging/VERSION
原来版本号在 4 个构建脚本各写死一个默认值 + openapi 里硬编码。
漏改一处就产出「文件名 1.0.9、control 里 1.0.8」的包 —— apt 按
control 认版本，dpkg -l 永远显示旧版且升不上去。
index.html 的 ETag 并入版本号：它发出去的是注入后的内容，
ETag 却按原文件算，只改 VERSION 文件时 ETag 不变，客户端换到 304
继续用旧副本，页面版本号永远停在旧版。
## logo 回概述
用 <a> 而不是 <div onclick>：Tab 能聚焦、Enter 能触发、读屏会念「链接」。
已在 dash 页时不重复 go()（go 会 stopPageTimers + 重拉数据）。
CSS 显式去 text-decoration/color（UA 样式表不声明仍生效），
:focus-visible 焦点环保留（纯键盘用户唯一的路标）。
## 顺带修掉一个已发布版本的真 bug
_vpn_env 里 os.path.join('/lib/modules', ...) 被留成了 '/nonexistent'
（反向验证脚本被 SIGTERM 打断，finally 没执行，注入留在源码里），
于是 mod_file 永远为空、**need_module 整个状态从四态里消失** ——
这正是 v1.0.8 release notes 声称修好的那个 bug（PVE 的 KVM 虚机内核
明明有 wireguard.ko.xz 却报 unsupported，前端配一句写死的
「通常说明跑在精简容器里」把排查方向带偏）。
新增 _dev/t-worktree-clean.py 扫注入哨兵，放在 sync.sh 所有判据之前 ——
基线被污染时，后面每一条红都不可信。
## 判据
新增 t-static-perf.py（42 项静态）、t-encoding-runtime.py（26 项**真跑**，
覆盖 9 种 Accept-Encoding 组合、brotli 缺失降级、两种编码不互相覆盖、
压不动直出、内部错误不被吞）、t-worktree-clean.py。
三个都做了反向验证，注入后确实变红。
反向验证本身也踩了坑：判据里 len(body) 在 body 为 None 时先崩 ——
退出码非 0 但没有 FAIL 行，看着像「跑了」，实际没人知道它验出了什么。
取长度一律过 _sz() 包一层。
全量回归 54 个判据 0 失败；真机验收 22 项全通过；
前端三查 47 视图 0 异常、26 项移动端适配全过。
|
2026-10-04 10:27:02 +08:00 |
|