代码审查¶
对未提交的更改执行全面的安全与质量审查:
-
查看已更改的文件列表:
git diff --name-only HEAD -
对每个已更改的文件,检查以下内容:
安全问题(CRITICAL): - 硬编码的凭据、API 密钥、令牌 - SQL 注入漏洞 - XSS 漏洞 - 缺少输入验证 - 不安全的依赖 - 路径遍历(Path Traversal)风险
代码质量(HIGH): - 超过 50 行的函数 - 超过 800 行的文件 - 超过 4 层的嵌套深度 - 缺少错误处理 - 调试日志语句(例如:开发用日志/print 等) - TODO/FIXME 注释 - 缺少活动语言的公共 API 文档(例如:JSDoc/Go doc/Docstring 等)
最佳实践(MEDIUM): - 变异(Mutation)模式(请使用不可变模式) - 在代码/注释中使用表情符号 - 缺少对新代码的测试 - 可访问性(a11y)问题
- 生成包含以下内容的报告:
- 严重程度:CRITICAL、HIGH、MEDIUM、LOW
- 文件位置及行号
- 问题描述
-
修复建议
-
如果发现 CRITICAL 或 HIGH 问题,则阻止 commit
绝不要批准存在安全漏洞的代码!
本页原文 Markdown:在 AtomGit 查看·内容源自开源项目 ev/ECC