security-review-skill-for-docker

name	security-review-skill-for-docker
description	审计 Docker/容器部署安全。检测 Dockerfile、docker-compose.yml、Kubernetes manifests 中的安全问题：特权容器、root 运行、敏感挂载、资源无限制、密钥泄露、Base Image 不合规、网络暴露等。当审计容器配置、Docker 安全、K8s 部署安全、或检查基础设施安全时使用。支持 Dockerfile、docker-compose.yml、Kubernetes YAML、Helm charts。

Docker / Container Security Review

适用范围

本 skill 审计以下文件类型的安全配置：

• Dockerfile / Dockerfile.*（多阶段构建）
• docker-compose.yml / docker-compose.*.yml
• Kubernetes manifests（Deployment, Pod, Service, Ingress, RBAC, NetworkPolicy）
• Helm charts（templates/ 下的 YAML）
• .dockerignore
• CI/CD 中的容器相关步骤

审计模式

触发 skill 后，根据用户 prompt 判断模式：

全量模式

扫描项目中所有容器相关文件。

PR/分支模式

git diff --name-only <base>...<target>

只审计变更的 Dockerfile / compose / K8s YAML 文件，但需 Read 完整文件理解上下文。

最近变更模式

git log --since="7 days ago" --name-only --pretty=format: | sort -u

筛选容器相关文件变更。

---

审计方法论

Step 1: 侦察 — 发现所有容器配置文件

# 发现 Dockerfile
Glob("**/Dockerfile*")

# 发现 docker-compose
Glob("**/docker-compose*.yml")
Glob("**/docker-compose*.yaml")

# 发现 K8s manifests
Glob("**/k8s/**/*.yml")
Glob("**/k8s/**/*.yaml")
Glob("**/deploy/**/*.yml")
Glob("**/manifests/**/*.yml")
Glob("**/helm/**/*.yaml")
Glob("**/.k8s/**/*.yaml")

# 发现 .dockerignore
Glob("**/.dockerignore")

# CI/CD 中的容器步骤
Glob("**/.github/workflows/*.yml")
Glob("**/.gitlab-ci.yml")
Glob("**/Jenkinsfile*")

Step 2: 正则扫描（必须执行）

对发现的所有文件，逐一执行以下正则检查：

Dockerfile 正则

# D1: Base Image 合规
Grep("^FROM ", glob="*Dockerfile*")                    # 所有 FROM 指令
Grep("FROM.*:latest", glob="*Dockerfile*")              # 禁止 :latest
Grep("FROM.*docker\\.io|FROM.*dockerhub", glob="*Dockerfile*")  # 禁止 Docker Hub 直拉
Grep("^FROM (?!jfrog\\.cmex\\.corp)", glob="*Dockerfile*")      # 必须用内部 registry

# D2: 权限控制
Grep("USER root", glob="*Dockerfile*")                  # 禁止 root 用户
Grep("--privileged", glob="*Dockerfile*")               # 禁止特权模式
Grep("chmod 777|chmod -R 777", glob="*Dockerfile*")     # 过宽文件权限
Grep("sudo ", glob="*Dockerfile*")                      # 容器内不应有 sudo

# D3: 密钥泄露
Grep("COPY.*\\.env|ADD.*\\.env", glob="*Dockerfile*")   # 禁止复制 .env
Grep("ARG.*PASSWORD|ARG.*SECRET|ARG.*TOKEN|ARG.*KEY", glob="*Dockerfile*", -i=true)  # ARG 传密钥
Grep("ENV.*PASSWORD|ENV.*SECRET|ENV.*TOKEN|ENV.*API_KEY", glob="*Dockerfile*", -i=true)  # ENV 硬编码
Grep("COPY.*\\.pem|COPY.*\\.key|COPY.*id_rsa", glob="*Dockerfile*")  # 复制私钥

# D4: 网络/SSH
Grep("EXPOSE 22|sshd|openssh", glob="*Dockerfile*")     # 禁止 SSH
Grep(, glob=)      


Grep(, glob=)                       
Grep(, glob=)  
Grep(, glob=)     
Grep(, glob=)  

docker-compose 正则

# D6: 特权和安全
Grep("privileged: true", glob="*docker-compose*")        # 特权容器
Grep("network_mode: host", glob="*docker-compose*")      # 主机网络
Grep("pid: host", glob="*docker-compose*")               # 主机 PID
Grep("ipc: host", glob="*docker-compose*")               # 主机 IPC
Grep("cap_add:", glob="*docker-compose*")                 # 能力添加（需逐一审查）

# D7: 资源限制
Grep("deploy:", glob="*docker-compose*")                  # 检查是否设了 resources.limits
Grep("mem_limit|memory:", glob="*docker-compose*")        # 内存限制

# D8: 敏感挂载
Grep("docker\\.sock", glob="*docker-compose*")            # Docker socket 挂载
Grep("/etc:/|/var:/|/root:/|/home:/", glob="*docker-compose*")  # 宿主机敏感路径
Grep("volumes:.*:rw", glob="*docker-compose*")            # 可写挂载（检查必要性）

# D9: 密钥
Grep("PASSWORD|SECRET|TOKEN|API_KEY", glob="*docker-compose*", -i=true)  # 明文密钥

Kubernetes 正则

# D10: Pod 安全
Grep("privileged: true", glob="**/*.{yml,yaml}")         # 特权容器
Grep("allowPrivilegeEscalation: true", glob="**/*.{yml,yaml}")
Grep("runAsUser: 0|runAsNonRoot: false", glob="**/*.{yml,yaml}")  # root 运行
Grep("hostNetwork: true|hostPID: true|hostIPC: true", glob="**/*.{yml,yaml}")
Grep("hostPath:", glob="**/*.{yml,yaml}")                 # 宿主机路径挂载

# D11: RBAC
Grep("cluster-admin|system:masters", glob="**/*.{yml,yaml}")  # 过宽权限
Grep("resources:.*\\*|verbs:.*\\*", glob="**/*.{yml,yaml}")   # 通配符权限

# D12: 资源限制
Grep("resources:", glob="**/*.{yml,yaml}")                # 检查 limits 存在
Grep("limits:", glob="**/*.{yml,yaml}")                   # CPU/memory limits

# D13: Secrets
Grep("kind: Secret", glob="**/*.{yml,yaml}")              # 明文 Secret（应用 sealed-secrets/external-secrets）
Grep("stringData:", glob="**/*.{yml,yaml}")               # 明文数据

Step 3: 逐文件深度审计

对每个发现的文件，Read 完整内容后按以下检查清单审计：

Dockerfile 审计清单

#	检查项	严重性	判定标准

| F1 | Base Image 来源 | HIGH | FROM 必须以 {{INTERNAL_REGISTRY}}/docker-local/ 开头 | | F2 | Image Tag 固定 | MEDIUM | 禁止 :latest，必须 pin 到具体版本或 sha256 digest | | F3 | Minimal Image | LOW | 优先 alpine/distroless/slim，非必要不用 ubuntu/debian full | | F4 | 非 Root 用户 | HIGH | 最终 stage 必须有 USER <非root>；中间 build stage 可以 root | | F5 | 无 COPY .env | CRITICAL | 禁止 COPY .env / ADD .env / COPY *.env | | F6 | 无硬编码密钥 | CRITICAL | ENV/ARG 不含密码/token/secret 的实际值（占位符可以） | | F7 | 无私钥复制 | CRITICAL | 禁止 COPY .pem/.key/id_rsa 到 final stage | | F8 | 无 SSH | HIGH | 禁止安装/启动 sshd | | F9 | COPY 优于 ADD | LOW | ADD 只应用于解压 tar，其他场景用 COPY | | F10 | Multi-stage build | LOW | 有编译步骤时应使用 multi-stage，final stage 不含编译工具 | | F11 | HEALTHCHECK | LOW | 建议有 HEALTHCHECK 指令 | | F12 | 层清理 | LOW | apt-get 后有 rm -rf /var/lib/apt/lists/*；apk add --no-cache | | F13 | 无 pipe to shell | HIGH | 禁止 curl|bash / wget|sh | | F14 | .dockerignore 存在 | MEDIUM | 项目根目录必须有 .dockerignore，排除 .git/.env/node_modules/*.key | | F15 | Build secrets 不泄露 | HIGH | Multi-stage 中 build stage 的 ARG/密钥不能出现在 final stage 的层中 |

docker-compose 审计清单

#	检查项	严重性	判定标准
C1	无 privileged	CRITICAL	禁止 privileged: true（除非有书面批准）
C2	最小 cap_add	HIGH	只允许: NET_BIND_SERVICE, NET_RAW（如需网络诊断）。SYS_ADMIN/ALL 禁止
C3	资源限制	HIGH	每个 service 必须有 deploy.resources.limits.memory 和 cpus
C4	无 host 网络	HIGH	禁止 network_mode: host（除 monitoring agent）
C5	Docker socket 审查	HIGH	/var/run/docker.sock 挂载必须注释说明原因，确认无替代方案
C6	无宿主机敏感路径	CRITICAL	禁止挂载 /etc、/var、/root、/home
C7	环境变量无明文密钥	HIGH	environment: 中密钥必须引用变量 ${VAR} 或 .env，不能明文写入
C8	端口暴露最小化	MEDIUM	仅暴露必要端口；内部服务不绑定到 0.0.0.0（用 127.0.0.1:port:port）
C9	restart 策略	LOW	使用 unless-stopped 或 on-failure，避免 always（防崩溃循环）
C10	read_only	LOW	建议 read_only: true + tmpfs 挂载运行时目录

Kubernetes 审计清单

#	检查项	严重性	判定标准
K1	securityContext	HIGH	Pod 必须有 securityContext.runAsNonRoot: true
K2	allowPrivilegeEscalation	HIGH	必须 false
K3	readOnlyRootFilesystem	MEDIUM	建议 true，用 emptyDir 做运行时写入
K4	资源 limits	HIGH	必须设 resources.limits.cpu 和 memory
K5	资源 requests	MEDIUM	必须设 resources.requests（调度保障）
K6	无 hostPath	HIGH	禁止 hostPath 挂载（用 PVC 替代）
K7	无 hostNetwork/PID/IPC	HIGH	禁止 hostNetwork/hostPID/hostIPC
K8	ServiceAccount	MEDIUM	automountServiceAccountToken: false（除非需要 API 访问）
K9	RBAC 最小化	HIGH	禁止绑定 cluster-admin；使用最小权限 Role
K10	Secret 管理	HIGH	禁止 kind: Secret + stringData 明文；用 sealed-secrets/external-secrets
K11	NetworkPolicy	MEDIUM	建议有 NetworkPolicy 限制 Pod 间通信
K12	Ingress 安全	MEDIUM	检查 TLS 配置、Host/Path 限制
K13	Image 来源	HIGH	同 F1，image: 必须用内部 registry
K14	Image tag	MEDIUM	同 F2，禁止 :latest

Step 4: 跨文件一致性检查

检查 Dockerfile 和 compose/K8s 之间的一致性：

1. Dockerfile 设了 USER 但 compose 用 user: root 覆盖 → CRITICAL
2. Dockerfile EXPOSE 的端口和 compose ports 不匹配 → INFO（可能是有意为之）
3. Multi-stage build 的 final stage image 是否和 compose/K8s 的 image 匹配
4. .dockerignore 是否排除了 .env，但 compose 中又 COPY 了 → 交叉检查

Step 5: 攻防验证

对每个发现的问题，验证：

• 攻击方：该配置是否真的可被利用？（例如：privileged: true 在隔离网络中风险降低但仍不推荐）
• 防御方：是否有其他层的防护？（例如：K8s PodSecurityPolicy/PodSecurityAdmission 可能已限制 privileged）

Step 6: 覆盖评估

#	维度	已覆盖?
D1	Base Image 合规（来源 + 版本）	[ ]
D2	权限控制（root/privileged/capabilities）	[ ]
D3	资源限制（memory/cpu）	[ ]
D4	网络安全（host network/端口暴露/SSH）	[ ]
D5	密钥管理（.env/硬编码/build args）	[ ]
D6	文件安全（.dockerignore/敏感挂载/volume）	[ ]
D7	Dockerfile 最佳实践（multi-stage/COPY/HEALTHCHECK）	[ ]
D8	K8s 安全上下文（securityContext/RBAC/SA）	[ ]
D9	供应链（image CVE/pipe-to-shell/untrusted sources）	[ ]

D1-D6 必须全部覆盖，D7-D9 建议覆盖。

---

误报过滤

Kill Switch 条件

场景	判定
USER root 在 multi-stage 的 非 final stage	安全（build stage 可以 root）
ADD 用于 .tar.gz 文件	安全（ADD 的合法用途）
cap_add: NET_RAW 用于网络诊断容器	降级为 LOW
docker.sock 挂载有注释说明且服务有访问控制	降级为 MEDIUM
privileged: true 在标注为 CI runner / 一次性 job 的容器	降级为 MEDIUM（仍需报告）
hostPath 用于 DaemonSet 的日志收集	降级为 LOW
cluster-admin 绑定到 CI/CD ServiceAccount 且有命名空间限制	降级为 MEDIUM
compose 中 environment: 的值是 ${VAR} 引用（非明文）	安全
ARG 用于 build 阶段且 final stage 不继承	安全

常见误报模式

1. Base image 为 FROM python:3.11-slim：Docker Hub 官方镜像。在内部 registry 未 mirror 的情况下，标记为 MEDIUM（建议迁移）而非 CRITICAL
2. EXPOSE 8080：EXPOSE 只是声明，不等于端口对外开放，需结合 compose/K8s 检查实际暴露
3. ENV APP_SECRET=changeme：如果是占位符/默认值且有文档说明需替换，降级为 LOW

---

公司特定规则

内部 Registry 规范

规则	要求
Base Image 来源	{{INTERNAL_REGISTRY}}/docker-local/ 或 {{INTERNAL_REGISTRY}}/docker-remote/（mirror）
禁止来源	直接 docker.io/、ghcr.io/、quay.io/（需通过内部 mirror）
Tag 格式	必须 pin 到版本号（如 python:3.11-slim），最佳实践是 digest（@sha256:...）

允许的 cap_add 白名单

Capability	允许场景	审查要求
NET_BIND_SERVICE	绑定低端口	LOW
NET_RAW	网络诊断/安全测试	需注释说明
SYS_PTRACE	调试容器（仅 dev）	禁止进入生产
其他任何	—	默认禁止，需安全团队审批

---

审计路由表

审计目标	读取文件
Dockerfile 安全	checks/dockerfile-security.md
docker-compose 安全	checks/compose-k8s-security.md
Kubernetes 部署安全	checks/compose-k8s-security.md
密钥和供应链	checks/secrets-supply-chain.md
全量审计	依次读取所有 checks/

已知问题对照：Read known-issues/common-vulnerabilities.md

---

输出格式

以 JSON 数组输出所有发现，放在 json ... 代码块中：

[
  {
    "title": "容器以 root 用户运行",
    "severity": "HIGH",
    "file": "Dockerfile",
    "line": 1,
    "description": "Dockerfile 缺少 USER 指令，容器默认以 root 运行。攻击者突破应用后直接获得 root 权限，可逃逸到宿主机。",
    "recommendation": "在最终 stage 添加非 root 用户：\nRUN addgroup -S app && adduser -S app -G app\nUSER app",
    "cwe_id": "CWE-250"
  }
]

每个 finding 字段：

• title：中文简述
• severity：CRITICAL / HIGH / MEDIUM / LOW
• file：文件路径
• line：行号
• description：漏洞原理和利用方式
• recommendation：修复建议（含代码示例）
• cwe_id：CWE 编号

重要：最后一条消息必须是完整的 findings JSON，不要在 JSON 之后输出其他内容。