跳转至

代码审查

对未提交的更改执行全面的安全与质量审查:

  1. 查看已更改的文件列表:git diff --name-only HEAD

  2. 对每个已更改的文件,检查以下内容:

安全问题(CRITICAL): - 硬编码的凭据、API 密钥、令牌 - SQL 注入漏洞 - XSS 漏洞 - 缺少输入验证 - 不安全的依赖 - 路径遍历(Path Traversal)风险

代码质量(HIGH): - 超过 50 行的函数 - 超过 800 行的文件 - 超过 4 层的嵌套深度 - 缺少错误处理 - 调试日志语句(例如:开发用日志/print 等) - TODO/FIXME 注释 - 缺少活动语言的公共 API 文档(例如:JSDoc/Go doc/Docstring 等)

最佳实践(MEDIUM): - 变异(Mutation)模式(请使用不可变模式) - 在代码/注释中使用表情符号 - 缺少对新代码的测试 - 可访问性(a11y)问题

  1. 生成包含以下内容的报告:
  2. 严重程度:CRITICAL、HIGH、MEDIUM、LOW
  3. 文件位置及行号
  4. 问题描述
  5. 修复建议

  6. 如果发现 CRITICAL 或 HIGH 问题,则阻止 commit

绝不要批准存在安全漏洞的代码!

本页原文 Markdown:在 AtomGit 查看·内容源自开源项目 ev/ECC