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 项移动端适配全过。
60 lines
2.7 KiB
Python
60 lines
2.7 KiB
Python
#!/usr/bin/env python3
|
|
# -*- coding: utf-8 -*-
|
|
"""响应压缩与协商缓存契约（#7）。
|
|
|
|
要挡住两类回归：
|
|
1. 压缩被悄悄去掉 —— 局域网里感觉不出来，一走外网（cloudflared / 远程）
|
|
就是 300KB 对 30KB 的差别；
|
|
2. 压缩后忘了发 Vary: Accept-Encoding —— 中间缓存会把 gzip 版喂给
|
|
不支持 gzip 的客户端，表现是「某些浏览器打开面板全是乱码」。
|
|
"""
|
|
import ast
|
|
import io
|
|
import os
|
|
import sys
|
|
|
|
ROOT = os.path.join(os.path.dirname(os.path.abspath(__file__)), '..')
|
|
fails = 0
|
|
|
|
|
|
def chk(label, cond, extra=''):
|
|
global fails
|
|
if not cond:
|
|
fails += 1
|
|
print('[%s] %s %s' % ('OK' if cond else 'FAIL', label, extra))
|
|
|
|
|
|
WEB = io.open(os.path.join(ROOT, 'backend/drouter-web.py'),
|
|
encoding='utf-8').read()
|
|
TREE = ast.parse(WEB)
|
|
|
|
chk('定义了压缩下限 GZIP_MIN', 'GZIP_MIN = ' in WEB)
|
|
chk('有 Accept-Encoding 判断', 'def _want_gzip' in WEB)
|
|
# ⚠️ 1.0.9 起压缩缓存从 _gz_cache/_br_cache 合并成一张 _comp_cache
|
|
# （键里带编码）。这里跟着改名，别断言一个已经不存在的变量 ——
|
|
# 判据断言错的名字比没有判据更糟：它会一直绿，而功能早就改了。
|
|
chk('压缩结果带内存缓存（不每次重压）',
|
|
'def _compress_cached' in WEB and '_comp_cache' in WEB)
|
|
chk('缓存按 mtime+size 失效（部署新版后自动重压）',
|
|
"int(st.st_mtime), st.st_size" in WEB)
|
|
chk('压缩后必须发 Content-Encoding', "send_header('Content-Encoding', enc)" in WEB)
|
|
chk('压缩后必须发 Vary: Accept-Encoding（否则代理会喂错客户端）',
|
|
WEB.count("send_header('Vary', 'Accept-Encoding')") >= 2,
|
|
'实际 %d 处' % WEB.count("send_header('Vary', 'Accept-Encoding')"))
|
|
chk('压缩比不划算时不压（原始数据更大就直出）', 'if len(z) >= len(data)' in WEB)
|
|
chk('静态资源支持 ETag 协商缓存', "send_header('ETag', etag)" in WEB)
|
|
chk('ETag 命中回 304 且不回传响应体',
|
|
"If-None-Match" in WEB and 'send_response(304)' in WEB)
|
|
# 调用点 1.0.9 起统一走 _encode_body（自动挑 br/gzip），
|
|
# 不再分别调 _gzip_cached —— 否则 br 永远没机会上场。
|
|
chk('JSON 接口走统一编码协商', '_encode_body(self.h.headers, None, body)' in WEB)
|
|
chk('静态资源走统一编码协商', '_encode_body(self.headers, full, data)' in WEB)
|
|
chk('HEAD 请求不回响应体', "if self.command != 'HEAD'" in WEB)
|
|
|
|
# 小响应不能被压：省下的字节还不如压缩开销
|
|
src = io.open(os.path.join(ROOT, 'backend/drouter-web.py'), encoding='utf-8').read()
|
|
chk('小响应不压缩（有体积下限判断）', 'len(data) >= GZIP_MIN' in src)
|
|
|
|
print('\n结果: %s' % ('全部通过' if fails == 0 else '%d 项失败' % fails))
|
|
sys.exit(1 if fails else 0)
|