# Security Review

> Use when reviewing code that touches auth, permissions, uploads, secrets, sandbox execution, logging, or before any public release. Do NOT use for purely cosmetic or docs-only changes.

- Skill: `photonics-dhl/security-review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add photonics-dhl/security-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/photonics-dhl/security-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Security
- Author: photonics-dhl (https://skillmd.com/u/photonics-dhl)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/photonics-dhl/security-review

---


# Security Review — 安全审查清单

安全审查逐条核对项，来源：Spec §17（安全与隐私）、§3.3（角色）、§10.3（沙箱限制）。

## 何时使用 / 何时不使用

- **使用**：涉及认证、权限、上传、密钥、沙箱、日志的代码评审；公开发布前检查；新增第三方依赖的安全评估。
- **不使用**：纯样式、文案、文档修改。

## 检查清单（Spec §17 MUST 项）

1. **密钥管理**：密钥仅来自服务器 Secret/环境变量，不进入仓库；代码中不得出现硬编码密钥；不得把 Secret 拉入 Agent 上下文（§17；§20.1-9）。
2. **越权检查在 API 层**：防止跨 Workspace 越权；每个受保护资源在 API 层校验调用者角色（Owner/Maintainer/Author/Contributor/Reviewer/Viewer/Moderator/Platform Admin，§3.3），禁止只靠前端隐藏。
3. **审计日志**：全部写操作记录审计日志。
4. **限流**：登录、发布、上传、AI、搜索和沙箱接口必须限流。
5. **上传安全**：上传文件进行类型检测、大小限制和恶意内容扫描（§17；配合 §13.1 分片、校验和、MIME 检测与病毒扫描）。
6. **公开前扫描**：内容公开前进行敏感信息扫描。
7. **Web 安全配置**：Session、Cookie、CSRF、CORS、CSP 正确配置；管理后台启用更强认证。
8. **日志脱敏**：生产日志不得记录完整论文、密钥、身份证信息或模型 Prompt 中的敏感附件。

## 沙箱专项（Spec §10.3 逐条核对）

9. **威胁模型单独维护**：Sandbox 威胁模型是独立文档，单独更新（§17）。
10. **资源限制**：30 秒、单核、1 GB 内存；进程数、文件数、输出大小受限。
11. **网络隔离**：禁止公网、内网、云元数据访问；沙箱容器不加入 `data_net`（§10.3、§14.2）。
12. **文件系统与权限**：非 root；只读根文件系统；临时目录；禁止宿主目录、Docker Socket 和数据库凭据。
13. **包与系统调用白名单**：仅 NumPy/SciPy/SymPy/Matplotlib/Pillow 等白名单包；禁止 `os`、`subprocess`、`socket`、`ctypes`、动态安装和任意二进制；执行完立即销毁容器。
14. **安全测试基线**：越权、上传、SSRF、Prompt Injection、Sandbox Escape 属 §21.1 安全测试层，发布前必须有对应测试证据。