liuzhuohua
ed5eb4428b
## 性能：瓶颈在传输，不在代码
先量后改。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 项移动端适配全过。
84 lines
3.3 KiB
Python
84 lines
3.3 KiB
Python
#!/usr/bin/env python3
|
||
# -*- coding: utf-8 -*-
|
||
"""开工前预检：工作区有没有「上一轮注入没还原」的残留。
|
||
|
||
为什么需要这个文件：反向验证（注入 bug → 看判据变红 → 还原）如果被
|
||
SIGTERM/SIGKILL 打断，`finally` 里的还原不会执行，注入就**留在源码里**。
|
||
下一次跑反向验证时，它看到的「基线」已经被污染，于是报出一堆看不懂的红，
|
||
而真正的根因（几小时前那次没还原的注入）完全不在视野里。
|
||
|
||
2026-10-04 实际踩到：`drouter-helper.py` 里 `_vpn_env` 的
|
||
`os.path.join('/lib/modules', ...)` 被留成了 `'/nonexistent'`，
|
||
连带 `need_module` 整个状态从四态里消失 —— 而这个缺陷在
|
||
**已发布的 v1.0.8 里**。t-107 一直红着，但被当成了「判据本身有问题」。
|
||
|
||
所以这一条不是「保险」，是真bug 的直接成因。
|
||
|
||
判据只查两个方向：
|
||
① backend/ 下有没有明显的注入哨兵（/nonexistent、__INJECT__、127.0.0.2）
|
||
② 全量静态判据当前是否全绿（不绿就别动手，先弄清是哪一条、为什么）
|
||
只读，不改任何东西。
|
||
"""
|
||
import io
|
||
import os
|
||
import re
|
||
import subprocess
|
||
import sys
|
||
|
||
ROOT = os.path.join(os.path.dirname(os.path.abspath(__file__)), '..')
|
||
# 注入脚本惯用的假路径 / 假常量。真机代码里出现它们基本可以断定是残留。
|
||
SENTINELS = ('/nonexistent', '__INJECT__', '127.0.0.2', 'INJECTED_')
|
||
|
||
fails = []
|
||
|
||
|
||
def chk(label, cond, extra=''):
|
||
print('[%s] %s %s' % ('OK' if cond else 'FAIL', label, extra))
|
||
if not cond:
|
||
fails.append(label)
|
||
|
||
|
||
print('=== 1. 扫 backend/ 里的注入哨兵 ===')
|
||
hits = []
|
||
for fn in sorted(os.listdir(os.path.join(ROOT, 'backend'))):
|
||
if not fn.endswith('.py'):
|
||
continue
|
||
p = os.path.join(ROOT, 'backend', fn)
|
||
try:
|
||
src = io.open(p, encoding='utf-8').read()
|
||
except Exception:
|
||
continue
|
||
# 只在**代码**里找，注释和 docstring 里为了说明历史 bug 常会写到
|
||
code = re.sub(r'#.*$', '', src, flags=re.M)
|
||
for s in SENTINELS:
|
||
if s in code:
|
||
for i, line in enumerate(src.splitlines(), 1):
|
||
if s in line and not line.strip().startswith('#'):
|
||
hits.append('%s:%d %s' % (fn, i, line.strip()[:70]))
|
||
chk('backend/ 里没有注入残留', not hits, '')
|
||
for h in hits:
|
||
print(' %s' % h)
|
||
if hits:
|
||
print(' ⚠ 这些多半是反向验证没还原。用 git diff 看清楚，'
|
||
'确认是残留就 git checkout 还原，**别直接改成新值** —— '
|
||
'新值可能才是 bug（/nonexistent 那次就差点被当成「本就该这样」）。')
|
||
|
||
print()
|
||
print('=== 2. backend/*.py 语法 ===')
|
||
bad = []
|
||
for fn in sorted(os.listdir(os.path.join(ROOT, 'backend'))):
|
||
if not fn.endswith('.py'):
|
||
continue
|
||
p = os.path.join(ROOT, 'backend', fn)
|
||
r = subprocess.run([sys.executable, '-m', 'py_compile', p],
|
||
capture_output=True)
|
||
if r.returncode:
|
||
bad.append('%s: %s' % (fn, r.stderr.decode('utf-8', 'replace')[:120]))
|
||
chk('backend/ 下所有模块语法正确', not bad, '')
|
||
for b in bad:
|
||
print(' %s' % b)
|
||
|
||
print()
|
||
print('\n结果: %s' % ('全部通过' if fails == 0 else '%d 项失败' % len(fails)))
|
||
sys.exit(1 if fails else 0)
|