代码评审的权限边界：三个最危险的默认值

同一个 PR 混入重构和功能改动，diff 会放大 2-3 倍

最小权限不是一句口号，是三个具体动作：降权运行、限定目录、收敛网络。

先说结论

把「它能碰到什么」缩到最小，剩下的漏洞才谈得上「影响有限」。

• 用独立低权限账号运行，别用 root
• 数据目录 750，含密钥的配置 600
• 只监听需要的地址，管理端口不对外
• 凭据不写进镜像与仓库
• 定期核对权限，别让临时改动留下来

三个默认值

这三个是审计里出现频率最高的：

• 以 root 运行：一个 RCE 就是整机失守
• 监听 0.0.0.0：管理接口直接暴露在公网
• 目录 777：任何进程都能改配置和数据库

怎么改

把格式问题交给 lint 后，人工评论里讲空格的占比能从三成降到 5% 以下。

这个服务的几个安全事实

和它自身机制相关的权限细节：

• 评审等待超过一天，上下文切换的成本会让作者宁愿推倒重写
• 把格式问题交给 lint 后，人工评论里讲空格的占比能从三成降到 5% 以下
• 评论要区分「必须改」和「可以讨论」，否则作者无法判断优先级
• 同一个 PR 混入重构和功能改动，diff 会放大 2-3 倍

凭据怎么放

评论要区分「必须改」和「可以讨论」，否则作者无法判断优先级。

能不用密码就不用：密钥、短期令牌、专用账号，三层叠加最省心。

自查清单

每月花十分钟做一遍：确认运行身份、检查监听端口、复核目录权限、清理临时账号、确认备份可恢复。

还有几处容易忽略的

都是小地方，但出问题时往往就卡在这里。

• 单次评审超过 400 行时缺陷发现率明显下降，超过 1000 行基本只剩风格意见
• 评审记录是团队知识的载体，三个月后翻记录比翻文档有效
• PR 攒了一周才提，两千行 diff 没人看得下去
• 只评论「这里可以更好」不说为什么，作者改不动
• 在 PR 里顺手重构无关文件，真正的改动被淹没

权限给多了不会立刻出事，但出事的时候一定是因为它。