liuzhuohua
79f24aa264
1.0.3 为纯修复版本，五个问题全部来自真机（192.168.7.3）深度排查：
翻 journal 与统一日志的 error 记录、全盘扫描文件名控制字符、把现行配置库
里每个模块的渲染结果重新交给对应真守护进程做语法校验。
- render(radvd): RDNSS 字段原先完全不校验（同段代码里 prefix 校验网段、
DNSSL 校验域名，唯独漏了它），填 IPv4 地址会原样写进 radvd.conf 导致
radvd 起不来，界面只回一句 radvd.conf:18 error: syntax error。
现逐项按 IPv6 地址校验并给中文提示；多值支持逗号/空格并统一按空格输出
（逗号形式 radvd 本就不认）；DNSSL 同样支持多域名；修正 DNSSL 生命期
误读 rdnss_life 的字段错位。
- cleanup: 自动清理的「每天 00:00」根本存不下来——前端 `Number(v) || 4`
与后端 `int(cfg.get('hour') or 4)` 都把 0 当成「没填」，叠加后保存即变
04:00 且改不回来。两处改为显式判断 None/''。执行时刻判定由「正好等于」
放宽为「不早于」，避免计划时刻恰好关机/重启导致当天彻底跳过。
- deploy.sh: 三处无条件覆写改为「只在文件不存在时初始化」——
active-theme（自定义主题每次部署被打回默认）、generated/isp-dns.conf
（PPPoE 上游 DNS 被擦、解析变空直到下次拨号）、cleanup enabled=true
（唯一会真删文件的模块被静默重新打开）。另补兜底：生效主题指向已不
存在的主题时回退 default 并打印说明。
- drouter-web: socketserver 默认 handle_error 把任何异常连整段 traceback
打进 journal，浏览器切页/关标签抛的 BrokenPipeError / ConnectionReset
属「对面先挂了」，三日攒数十条足以淹没真报错。新增 _Server 子类只静默
这几类，其余照旧抛出。
- devtools: 新增 fix-crlf-names.py 清理 CRLF 环境残留的名字带 \r 脏条目
（保守：默认只预演、仅删有同名兄弟者、目录需相对文件集合为子集、
跳过 snapshots/）；sync.sh 发布预检增加「文件名不得含控制字符」。
184 lines
9.2 KiB
Python
184 lines
9.2 KiB
Python
# -*- coding: utf-8 -*-
|
||
"""read_render_text 逻辑测试（本地模拟，helper 因 pwd 无法在 Windows 导入）。"""
|
||
import sys, re, io
|
||
sys.path.insert(0, r"C:/Users/lyrz-pve-win10/WorkBuddy/2026-09-27-19-57-00/router-build/backend")
|
||
import render
|
||
|
||
src = io.open(r"C:/Users/lyrz-pve-win10/WorkBuddy/2026-09-27-19-57-00/router-build/backend/drouter-helper.py",
|
||
encoding='utf-8').read()
|
||
m = re.search(r'^def read_render_text\(.*?(?=\n\ndef act_apply)', src, re.S | re.M)
|
||
assert m, 'read_render_text 未找到'
|
||
NS = {'render': render, 'ValidateError': render.ValidateError, 'str': str}
|
||
exec('def ok(data=None, msg="ok", code="OK"):\n return {"ok": True, "code": code, "msg_cn": msg, "data": data}\n', NS)
|
||
exec('def fail(msg, code="ERR", data=None):\n return {"ok": False, "code": code, "msg_cn": msg, "data": data}\n', NS)
|
||
exec(m.group(0), NS)
|
||
fn = NS['read_render_text']
|
||
|
||
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))
|
||
|
||
cfg = {'wan_iface': 'ppp0', 'lan_ifaces': ['ens18'],
|
||
'portfwd': {'enable': True, 'rules': [{'name': 'NAS', 'proto': 'tcp', 'ext_port': 5000,
|
||
'int_ip': '192.168.7.10', 'int_port': 5000, 'enable': True}],
|
||
'dmz': {'enable': True, 'host': '192.168.7.50'}}}
|
||
r = fn({'module': 'nft_v4', 'cfg': cfg})
|
||
chk('nft_v4 预览成功', r['ok'])
|
||
lines = [x for x in r['data']['text'].split('\n') if 'dnat' in x or '放通转发' in x]
|
||
for x in lines: print(' ', x.strip())
|
||
chk('含 dnat 规则', any('dnat to 192.168.7.10' in x for x in lines))
|
||
chk('含 DMZ 规则', any('DMZ' in x for x in lines))
|
||
chk('含放通规则', any('放通转发' in x for x in lines))
|
||
|
||
r2 = fn({'module': 'nft_v4', 'cfg': {'wan_iface': 'ppp0', 'lan_ifaces': ['ens18'],
|
||
'portfwd': {'enable': True, 'rules': [{'name': 'x', 'proto': 'tcp', 'ext_port': 80,
|
||
'int_ip': '1.2.3.999', 'int_port': 80}]}}})
|
||
chk('非法 IPv4 返回中文错误', (not r2['ok']) and 'IPv4' in r2['msg_cn'], '→ ' + r2['msg_cn'])
|
||
|
||
r3 = fn({'module': 'dnsmasq', 'cfg': {'lan_iface': 'ens18', 'domain': 'lan'}})
|
||
chk('dnsmasq 预览成功', r3['ok'] and len(r3['data']['text']) > 50,
|
||
'长度 %s' % (len(r3['data']['text']) if r3['ok'] else '-'))
|
||
|
||
r4 = fn({'module': 'radvd', 'cfg': {'iface': 'ens18', 'prefix': '2408:8207:1234::/64'}})
|
||
chk('radvd 预览成功', r4['ok'] and r4['data']['text'])
|
||
|
||
r5 = fn({})
|
||
chk('空模块名被拒绝', (not r5['ok']) and r5['code'] == 'NO_MODULE', '→ ' + r5['msg_cn'])
|
||
|
||
r6 = fn({'module': 'nonsense'})
|
||
chk('未知模块返回中文错误', (not r6['ok']) and '未知' in r6['msg_cn'], '→ ' + r6['msg_cn'])
|
||
|
||
# ---------------------------------------------------------------- dnsmasq
|
||
# code 3 / 6 有专属字段，但「自定义 Options」表也能写同一个 code。
|
||
# 1.0.1 及更早会渲染成**重复的 dhcp-option 行**（dnsmasq 对 3/6 是追加语义），
|
||
# 客户端因此收到重复值、DNS 顺序也被搅乱。这里锁死合并后的行为。
|
||
|
||
def opt_lines(text, code):
|
||
"""取某个 code 的**非 force** dhcp-option 行的值"""
|
||
return re.findall(r'^dhcp-option=%s,(.*)$' % code, text, re.M)
|
||
|
||
|
||
def dup_codes(text):
|
||
"""被重复下发的 code（只算非 force 行；force 行与普通行共存是合法的）"""
|
||
codes = re.findall(r'^dhcp-option=(\d+),', text, re.M)
|
||
return sorted({c for c in codes if codes.count(c) > 1})
|
||
|
||
|
||
print('\n--- dnsmasq：code 3 / 6 不得渲染出重复行（1.0.2）---')
|
||
|
||
# 用户真机上那份配置：专属字段与自定义表都写了 3/6
|
||
CFG_DUP = {
|
||
'lan_iface': 'ens18', 'domain': 'lan', 'dhcp_enabled': True,
|
||
'pool_start': '192.168.7.1', 'pool_end': '192.168.7.154',
|
||
'pool_netmask': '255.255.255.0', 'lease_time': 43200,
|
||
'option_gateway': '192.168.7.3',
|
||
'option_dns': '223.5.5.5,119.29.29.29',
|
||
'options': [
|
||
{'enabled': True, 'code': '6', 'value': '192.168.7.3'},
|
||
{'enabled': True, 'code': '3', 'value': '192.168.7.3'},
|
||
],
|
||
'dns_mode': 'custom', 'dns_custom': '223.5.5.5,119.29.29.29',
|
||
}
|
||
t = render.render_dnsmasq(CFG_DUP)
|
||
for _l in t.split('\n'):
|
||
if _l.startswith('dhcp-'):
|
||
print(' ', _l)
|
||
chk('没有任何 code 被重复下发', dup_codes(t) == [], str(dup_codes(t)))
|
||
chk('code 3 只有一行', len(opt_lines(t, 3)) == 1, str(opt_lines(t, 3)))
|
||
chk('code 3 的值已去重', opt_lines(t, 3) == ['192.168.7.3'], str(opt_lines(t, 3)))
|
||
chk('code 6 只有一行', len(opt_lines(t, 6)) == 1, str(opt_lines(t, 6)))
|
||
chk('code 6 合并了自定义值', opt_lines(t, 6) == ['223.5.5.5,119.29.29.29,192.168.7.3'],
|
||
str(opt_lines(t, 6)))
|
||
|
||
# 专属字段留空时，自定义表里的 3/6 不能被吞掉
|
||
t2 = render.render_dnsmasq(dict(CFG_DUP, option_gateway='', option_dns='',
|
||
options=[{'enabled': True, 'code': '3', 'value': '192.168.7.1'}]))
|
||
chk('专属字段为空时自定义 3 仍输出', opt_lines(t2, 3) == ['192.168.7.1'], str(opt_lines(t2, 3)))
|
||
chk('专属字段为空时不下发 code 6', opt_lines(t2, 6) == [], str(opt_lines(t2, 6)))
|
||
|
||
# 去重要保序：先出现的先保留
|
||
t3 = render.render_dnsmasq(dict(CFG_DUP, option_dns='223.5.5.5,1.1.1.1',
|
||
options=[{'enabled': True, 'code': '6', 'value': '1.1.1.1,8.8.8.8'}]))
|
||
chk('顺序去重且保留首次出现顺序', opt_lines(t3, 6) == ['223.5.5.5,1.1.1.1,8.8.8.8'],
|
||
str(opt_lines(t3, 6)))
|
||
|
||
# 带「强制」的 3/6 语义不同（客户端未请求也要下发），不能并进普通行
|
||
t4 = render.render_dnsmasq(dict(CFG_DUP, options=[
|
||
{'enabled': True, 'code': '3', 'value': '192.168.7.9', 'force': True},
|
||
{'enabled': True, 'code': '3', 'value': '192.168.7.9'},
|
||
]))
|
||
chk('force 的 3 走 dhcp-option-force', 'dhcp-option-force=3,192.168.7.9' in t4)
|
||
chk('非 force 的 3 合并成一行', opt_lines(t4, 3) == ['192.168.7.3,192.168.7.9'],
|
||
str(opt_lines(t4, 3)))
|
||
|
||
# 其它 code 必须完全不受影响
|
||
t5 = render.render_dnsmasq(dict(CFG_DUP, options=[
|
||
{'enabled': True, 'code': '43', 'value': 'abc.example.com'},
|
||
{'enabled': True, 'code': '121', 'value': '10.0.0.0/8,192.168.7.3'},
|
||
{'enabled': True, 'code': '121', 'value': '1.2.3.0/24,192.168.7.3', 'force': True},
|
||
]))
|
||
chk('其它 code 原样输出', 'dhcp-option=43,abc.example.com' in t5)
|
||
chk('其它 code 的 force 保留', 'dhcp-option-force=121,1.2.3.0/24,192.168.7.3' in t5)
|
||
chk('121 两行是允许的（值不同）', len(re.findall(r'^dhcp-option(?:-force)?=121,', t5, re.M)) == 2)
|
||
|
||
# 预览接口走的是 helper 的 read_render_text，必须与直接渲染一致
|
||
p = fn({'module': 'dnsmasq', 'cfg': CFG_DUP})
|
||
chk('预览路径同样不重复', p['ok'] and dup_codes(p['data']['text']) == [],
|
||
str(dup_codes(p['data']['text'])) if p['ok'] else p.get('msg_cn'))
|
||
chk('预览里 code 3 也是一行', p['ok'] and opt_lines(p['data']['text'], 3) == ['192.168.7.3'],
|
||
str(opt_lines(p['data']['text'], 3)) if p['ok'] else '-')
|
||
|
||
# 校验不能被放松
|
||
try:
|
||
render.render_dnsmasq(dict(CFG_DUP, option_gateway='1.2.3.999'))
|
||
chk('非法网关仍被拒绝', False, '居然没抛异常')
|
||
except render.ValidateError as ex:
|
||
chk('非法网关仍被拒绝', 'IPv4' in str(ex), str(ex))
|
||
|
||
# ---------------------------------------------------------------- radvd RDNSS
|
||
# RA 通告里根本没有 IPv4 的位置：RDNSS 只能是 IPv6 地址，且多值必须**空格分隔**
|
||
# （逗号形式 radvd 直接 syntax error）。这里原先完全不校验，于是「下发 DNS」
|
||
# 框里顺手填个 223.5.5.5 就会渲染出一份 radvd 读不进去的配置 ——
|
||
# 真机上留下的错误日志就是 "/run/drouter-verify/radvd.conf:18 error: syntax error"，
|
||
# 而界面只回一句英文，完全指不到是哪个字段。
|
||
|
||
def ra(extra):
|
||
return render.render_radvd(dict({'iface': 'ens18', 'prefix': 'fd00:7::/64'}, **extra))
|
||
|
||
for _bad in ('223.5.5.5', '223.5.5.5,119.29.29.29', 'dns.alidns.com',
|
||
'fd00:7::/64'):
|
||
try:
|
||
ra({'rdnss': _bad})
|
||
chk('RDNSS 拒绝 %r' % _bad, False, '居然通过了')
|
||
except render.ValidateError as ex:
|
||
chk('RDNSS 拒绝 %r' % _bad, 'IPv6' in str(ex), str(ex))
|
||
|
||
_r = ra({'rdnss': 'fd00:7::1 fd00:7::2'})
|
||
chk('RDNSS 多个 IPv6 合并成一行（空格分隔）',
|
||
re.search(r'^\s*RDNSS fd00:7::1 fd00:7::2 \{$', _r, re.M) is not None)
|
||
_r = ra({'rdnss': 'fd00:7::1,fd00:7::2'})
|
||
chk('RDNSS 逗号列表归一成空格（而不是让 radvd 报 syntax error）',
|
||
re.search(r'^\s*RDNSS fd00:7::1 fd00:7::2 \{$', _r, re.M) is not None)
|
||
|
||
_r = ra({'rdnss': 'fd00:7::1', 'dnssl': 'lan,home.lan'})
|
||
chk('DNSSL 支持多域名（空格分隔）',
|
||
re.search(r'^\s*DNSSL lan home\.lan \{$', _r, re.M) is not None)
|
||
try:
|
||
ra({'rdnss': 'fd00:7::1', 'dnssl': 'ok.lan,bad@lan'})
|
||
chk('DNSSL 非法域名被拒绝', False, '居然通过了')
|
||
except render.ValidateError as ex:
|
||
chk('DNSSL 非法域名被拒绝', 'DNSSL' in str(ex), str(ex))
|
||
|
||
# DNSSL 生命期原来读的是 rdnss_life —— 「DNSSL 生命期」这个 label 指向别人的值
|
||
chk('DNSSL 生命期优先取 dnssl_life',
|
||
'AdvDNSSLLifetime 1200;' in ra({'rdnss': 'fd00:7::1', 'dnssl': 'lan',
|
||
'dnssl_life': 1200}))
|
||
chk('没有 dnssl_life 时退回 rdnss_life',
|
||
'AdvDNSSLLifetime 900;' in ra({'rdnss': 'fd00:7::1', 'dnssl': 'lan',
|
||
'rdnss_life': 900}))
|
||
|
||
print('\n结果: %s' % ('全部通过' if fails == 0 else '%d 项失败' % fails))
|
||
sys.exit(1 if fails else 0)
|