security-review

原文涵盖的主题

• 安全审查技能
• 何时激活
• 安全检查清单
• 1. 密钥管理
• FAIL: 绝对不要这样做
• PASS: 始终这样做
• 验证步骤
• 2. 输入验证

此技能确保所有代码遵循安全最佳实践，并识别潜在漏洞。 实现身份验证或授权时 处理用户输入或文件上传时 创建新的 API 端点时 处理密钥或凭据时 实现支付功能时 存储或传输敏感数据时 集成第三方 API 时 \[ ] 没有硬编码的 API 密钥、令牌或密码 \[ ] 所有密钥都存储在环境变量中 \[ ] .env 文件在 .gitignore 中 \[ ] git 历史记录中没有密钥 \[ ] 生产环境密钥存储在托管平台中（Vercel, Railway） \[ ] 所有用户输入都使用模式进行了验证 \[ ] 文件上传受到限制（大小、类型、扩展名） \[ ] 查询中没有直接使用用户输入 \[ ] 使用白名单验证（而非黑名单） \[ ] 错误消息不会泄露敏感信息 \[ ] 所有数据库查询都使用参数化查询 \[ ] SQL 中没有字符串拼接 \[ ] 正确使用 ORM/查询构建器 \[ ] Supabase 查询已正确清理 \[ ] 令牌存储在 httpOnly cookie 中（而非 localStorage） \[ ] 执行敏感操作前进行授权检查 \[ ] Supabase 中启用了行级安全 \[ ] 实现了基于角色的访问控制 \[ ] 会话管理安全 \[ ] 用户提供的 HTML 已被清理 \[ ] 已配置 CSP 头部 \[ ] 没有渲染未经验证的动态内容 \[ ] 使用了 React 内置的 XSS 防护 \[ ] 状态变更操作上使用了 CSRF 令牌 \[ ] 所有 Cookie 都设置了 SameSite=Strict \[ ] 实现了双重提交 Cookie 模式 \[ ] 所有 API 端点都实施了速率限制 \[ ] 对昂贵操作有更严格的限制 \[ ] 基于 IP 的速率限制 \[ ] 基于用户的速率限制（已认证） \[ ] 日志中没有密码、令牌或密钥 \[ ] 对用户显示通用错误消息 \[ ] 详细错误信息仅在服务器日志中 \[ ] 没有向用户暴露堆栈跟踪 \[ ] 已验证钱包签名 \[ ] 已验证交易详情 \[ ] 交易前检查余额 \[ ] 没有盲签名交易 \[ ] 依赖项是最新的 \[ ] 没有已知漏洞（npm audit 检查通过） \[ ] 提交了锁定文件 \[ ] GitHub 上启用了 Dependabot \[ ] 定期进行安全更新 在任何生产环境部署前： \[ ] 密钥：没有硬编码的密钥，全部在环境变量中 \[ ] 输入验证：所有用户输入都已验证 \[ ] SQL 注入：所有查询都已参数化 \[ ] XSS：用户内容已被清理 \[ ] CSRF：已启用防护 \[ ] 身份验证：正确处理令牌 \[ ] 授权：已实施角色检查 \[ ] 速率限制：所有端点都已启用 \[ ] HTTPS：在生产环境中强制执行 \[ ] 安全头部：已配置

这里只展示节选，请以完整原文和当前要求为准。