fix(deploy): 环境兜底自己依赖的命令也登记 + 自我检查排在用它们之前 + 静默改 HOME 必须留痕
pi 评审 2026-09-15 报的"第五次环境假设",在 `env-defaults.sh` **自己**身上。 他指出的**结构**成立:本文件用了 `id`/`getent`/`cut`/`df`/`awk`,一个都没登记进 `AGENTMAIL_REQUIRE`(那张表只登记**调用者**的命令,且由调用者在**source 之后**赋值)。 ★ 但我实测发现**他给的两个具体后果在这台机器上不可达**,原因值得记下来: `env-defaults.sh` 的 ④ PATH 自修(`:46`)在 PATH 里没有 `/usr/bin` 时会**把它加回来** ⇒ "从 PATH 里拿掉 id/getent/cut/df/awk"这种造法**必然被自修抵消**(我第一版探针就栽在这里: `id -u` 根本没失败,我却按"失败了"往下推理,直到把 `command -v id` 单独打出来才看见)。 缺这些命令只可能发生在"**`/usr/bin` 里真没有它**"的机器上(distroless / 精简容器)。 所以这次修的是**能 durable 判定的三件**,而不是他描述的失败面: 1. **登记**:新增文件级常量 `AGENTMAIL_REQUIRE_SELF="id getent cut df awk"`。 为什么不写进三个调用者的 `AGENTMAIL_REQUIRE`:那个变量在 source 时**还不存在** (`. env-defaults.sh` 在第 16/42/52 行,`AGENTMAIL_REQUIRE=` 在第 20/46/56 行), 本文件没法把它自己那份追加进一个"稍后才被赋值"的变量 —— 追加了本次也不生效。 2. **自我检查排在用它们之前**(顺序即正确性,同 ①→④ 那条):新增 ③b-0 段, 只用了**内建命令**(`command -v` + `printf`),所以能在"环境还什么都没兜"时跑; 它现在位于 `:76`,而第一次真正用这些命令的 `id -u` 在 `:102`。 ⇒ 缺 `df`/`awk` 时**不再静默丢门**:原来 `df -Pk … | awk` 拿到空串会落进 `''|*[!0-9]*)` 那支"读不到 ⇒ 不判定",**②b 那道空间门直接消失**(那是门,不是提示)。 3. **静默改 HOME 必须留痕**:原先只在"**调用者给的** HOME 不可写"时 WARN, 而"按身份推出来的那个也不可用"(root 的 `/root` 在非 root 下不可写; passwd 里是 `/nonexistent`)**悄悄换了 HOME** —— 与本文件存在的理由正好相反。 现在两条路都 WARN。★ 这一条**可达且实测过**: `setpriv --reuid=65534 … bash -c 'unset HOME; source env-defaults.sh'` ⇒ `[WARN] 按身份推出来的 HOME=/nonexistent 不可用 … 改判到 /tmp/agentmail-home-65534`。 **判据 4 条**(`test/env-guard.test.mjs`,pi 桥侧,与该文件既有的环境判据同处): ① `AGENTMAIL_REQUIRE_SELF` 登记了这 5 个命令;② **顺序**:自我检查的行号必须**小于** `id -u` 的行号(判据写成位置比较,而不是"有这段代码" —— 后者正是我这一轮反复写坏的形状); ③ 源码里存在"按身份推出来的 HOME 不可用"那句 WARN;④ **端到端**:非 root + 空 HOME 真的打出 WARN。 ★ 这条端到端判据我写坏了**两次**,都记在文件里: · 第一版用 `execFileSync` 只收 stdout,而 WARN 走 **stderr** ⇒ 红在"没找到 WARN"上, 实际是**判据自己没读那一股**; · 改用 `spawnSync` 后仍红 —— 因为 `deploy/lib/env-defaults.sh` 是 **0600**, `nobody` 读不到它,脚本**压根没跑起来**。这与"命令不在 ≠ 输出为空"是同族: **脚本没跑 ≠ 输出里没有那一行**。判据改为用一份世界可读的副本(文件权限是另一件事)。 ⇒ 顺带发现并修掉:我用写文件工具建的 5 个文件都是 **0600**(该工具不理会 umask), 已全部改 644(仓库既有约定;同目录其他文件都是 644/755)。 **`cp -a` 会把 0600 带进生产快照**,所以这不是纯本地问题 —— 记一笔,未另开检查 (工作区里还有 52 个 git 已跟踪文件是 0600,是既有状态、非本次引入,单独处理)。 验证:pi 桥 **509/509**(+4);`check-shared-libs` exit 0;`install.sh --check` exit 0。
This commit is contained in:
@ -22,7 +22,8 @@
|
||||
|
||||
import assert from 'node:assert/strict';
|
||||
import test from 'node:test';
|
||||
import { execFileSync } from 'node:child_process';
|
||||
import { execFileSync, spawnSync } from 'node:child_process';
|
||||
import { readFileSync, writeFileSync, unlinkSync } from 'node:fs';
|
||||
// (本文件不再直接读文件:夹具在 test/lib/,边界判据在 layout-boundaries.test.mjs)
|
||||
import { fileURLToPath } from 'node:url';
|
||||
import { dirname, join } from 'node:path';
|
||||
@ -220,3 +221,73 @@ test('接线:writeSession 撞上 ENOSPC 时抛出的必须是翻译过的错
|
||||
);
|
||||
});
|
||||
|
||||
|
||||
|
||||
// ── deploy/lib/env-defaults.sh 自身的外部命令依赖(pi 评审 2026-09-15)──
|
||||
|
||||
const ENV_DEFAULTS = join(HERE, '..', '..', '..', 'deploy', 'lib', 'env-defaults.sh');
|
||||
|
||||
test('★ 环境兜底文件必须把**自己**用的外部命令登记出来', () => {
|
||||
// pi 评审 2026-09-15:`env-defaults.sh` 自己用了 `id`/`getent`/`cut`/`df`/`awk`,
|
||||
// 一个都没进 `AGENTMAIL_REQUIRE`(那张表只登记**调用者**的命令,而它由调用者赋值)。
|
||||
// ⇒ "环境兜底自己还需要环境",且缺 `df`/`awk` 时 **②b 那道空间门静默消失**
|
||||
// (`df -Pk … | awk` 拿到空串 ⇒ 走"读不到 ⇒ 不判定"那支)。
|
||||
//
|
||||
// ★ 为什么判据落在"登记了没有"而不是"缺命令会 exit 2":
|
||||
// 我实测过——`env-defaults.sh` 的 **PATH 自修**(④)会把 `/usr/bin` 加回来,
|
||||
// 所以"从 PATH 里拿掉 df"这种造法**根本造不出缺命令的环境**(自修又把它找回来了)。
|
||||
// 缺命令只可能发生在"`/usr/bin` 里真没有它"的机器上(distroless / 精简容器),
|
||||
// 而那种环境我在这台机器上无法复现。⇒ 能**durable** 判定的只有两件事:
|
||||
// ①这些命令被登记了;②检查发生在**下一次外部调用之前**。两条都断言。
|
||||
const src = readFileSync(ENV_DEFAULTS, 'utf8');
|
||||
const selfReq = src.match(/AGENTMAIL_REQUIRE_SELF="([^"]*)"/);
|
||||
assert.ok(selfReq, 'env-defaults.sh 必须声明它自己依赖的命令(AGENTMAIL_REQUIRE_SELF)');
|
||||
for (const c of ['id', 'getent', 'cut', 'df', 'awk']) {
|
||||
assert.ok(selfReq[1].split(/\s+/).includes(c), `AGENTMAIL_REQUIRE_SELF 里应当登记 ${c}`);
|
||||
}
|
||||
});
|
||||
|
||||
test('★ 顺序即正确性:自我依赖的检查必须在**第一次用它们之前**', () => {
|
||||
// 这是 pi 那条"顺序不是风格,是正确性"的口径落在**本文件自己**身上。
|
||||
// 判据写成位置比较,而不是"有这段代码"——后者正是我这一轮反复写坏的那种判据。
|
||||
const src = readFileSync(ENV_DEFAULTS, 'utf8');
|
||||
const lines = src.split('\n');
|
||||
const checkLine = lines.findIndex((l) => l.includes('_am_self_missing='));
|
||||
const firstUseLine = lines.findIndex((l) => /^_am_uid="\$\(id /.test(l));
|
||||
assert.ok(checkLine > 0, '应当有自我依赖检查');
|
||||
assert.ok(firstUseLine > 0, '应当有 `id -u` 那次使用');
|
||||
assert.ok(checkLine < firstUseLine,
|
||||
`自我依赖检查必须排在 \`id -u\` 之前(实际 ${checkLine + 1} vs ${firstUseLine + 1})`);
|
||||
});
|
||||
|
||||
test('★ 静默改 HOME 必须留痕(兜底自己被环境打败那条路)', () => {
|
||||
// 原先只有"调用者给的 HOME 不可写"才 WARN;而"按身份推出来的那个也不可用"
|
||||
// (root 的 /root 在非 root 下不可写;passwd 里是 /nonexistent)**悄悄换了 HOME**。
|
||||
// 实测可达:非 root + 空 HOME ⇒ nobody 的 passwd home 是 /nonexistent。
|
||||
const src = readFileSync(ENV_DEFAULTS, 'utf8');
|
||||
assert.match(src, /按身份推出来的 HOME=%s 不可用/,
|
||||
'兜底路径改了 HOME 就必须打一行 WARN("东西写到哪去了"不能变成谜)');
|
||||
});
|
||||
|
||||
test('非 root + 空 HOME ⇒ 改判 HOME 时**确实**会打 WARN(端到端)', (t) => {
|
||||
if (process.getuid?.() !== 0) return t.skip('需要 root 才能切到 nobody 复现');
|
||||
// ★ 必须**同时**收 stderr:WARN 走 stderr(`>&2`),只收 stdout 会把它丢掉 ——
|
||||
// 我第一版就是这样,于是判据红在"没找到 WARN"上,而实际是**判据自己没去读那一股**。
|
||||
// 用一个世界可读的副本:仓库里的 `deploy/lib/env-defaults.sh` 可能是 0600
|
||||
// (我在 umask 077 下用写文件工具建过),而 `nobody` 读不了它 —— 那是**另一件事**
|
||||
// (文件权限),不该混进"改判 HOME 是否留痕"这条判据里。
|
||||
// ★ 顺带记下:这条判据第一次就是**红在这个权限上**,报的是"没找到 WARN",
|
||||
// 看起来像"没打 WARN",实际是脚本**根本没跑起来**。这正是"命令不在≠输出为空"
|
||||
// 的同族:**脚本没跑 ≠ 输出里没有那一行**。
|
||||
const staged = join(tmpdir(), `am-env-defaults-${process.pid}.sh`);
|
||||
writeFileSync(staged, readFileSync(ENV_DEFAULTS, 'utf8'), { mode: 0o644 });
|
||||
const r = spawnSync('setpriv', ['--reuid=65534', '--regid=65534', '--clear-groups',
|
||||
'env', '-i', 'PATH=/usr/bin:/bin', 'TMPDIR=/tmp',
|
||||
'bash', '-c', `unset HOME; source ${staged}; echo "FINAL=$HOME"`],
|
||||
{ encoding: 'utf8' });
|
||||
try { unlinkSync(staged); } catch { /* 清理失败不影响判据 */ }
|
||||
const out = `${r.stdout || ''}${r.stderr || ''}`;
|
||||
assert.match(out, /\[WARN\]/, '改判 HOME 必须留下 WARN');
|
||||
assert.match(out, /FINAL=\/tmp\/agentmail-home-65534|FINAL=\/root/,
|
||||
`最终 HOME 要说得出落在哪,实际输出:${out}`);
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user