首页/开发/Git 与协作速查/代码评审中的安全要点
协作与代码评审75 条

代码评审中的安全要点

评审时优先看四类高风险面:输入校验与注入、认证授权、密钥与敏感数据处理、依赖引入;发现问题按私下渠道报告而不是公开评论。

Git 与协作速查开发更新于 2026-10-12
输入处理走查询参数/命令/模板的输入是否校验与转义
权限接口是否有鉴权与越权校验(横向与纵向)
密钥是否把令牌、证书、内网地址写进代码或日志
依赖新增依赖的来源、版本与许可证是否合规

快速自查命令

快速自查命令
git diff origin/main...HEAD --stat
git diff origin/main...HEAD | grep -nE '(password|token|secret|apikey|BEGIN [A-Z ]*PRIVATE KEY)' || true
扫描只能兜底:真正的判断点是这段代码在异常输入与恶意使用者面前是否安全。

易错点

  • 把疑似漏洞细节写进公开 PR 评论会扩大暴露面,先私下联系维护者
  • 只依赖自动扫描通过就放行,逻辑层越权类问题通常扫不出来

数据来源与参考

  • GitHub Docs:Reviewing security-related changes(代码扫描与依赖告警)
  • GitLab Docs:Security in merge requests
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌