代码评审问题及报告
代码平评审问题报告
评审对象:评审本次需求上线新增的类
评审内容概述:对后端代码的业务逻辑、垂直和水平越权等问题作出评审
评审功能详述:某某某一些功能
是否涉及关键代码:是,(文件上传、短信通知)
本系统代码评审要点:
序号 | 代码评审检查类别 | 检查内容 |
1 | 业务逻辑检查 | 代码的逻辑检查,尤其针对if判断 |
2 | SQL语句检查 | insert语句 |
3 | 异常处理检查 |
自定义语异常的抛出
|
4 | 垂直、水平越权漏洞代码安全走查 | 检查各接口是否有垂直 、水平越权效验 |
5 | 错误验证检查 | 方法的参数是否都做了效验 |
本次评审发现的问题:
一般问题 2 轻微问题2
序号 | 问题描述 | 解决建议 | 提出人 | 严重程度 | 解决方式 |
1 | 某某接口/未做水平越权 | 需要校验 | 某某 | 一般 | 做村服绑定校验 |
2 | 某某接口/未对渠道权限进行校验,未对入参进行效验 | 需要校验 | 某某 | 一般 |
对渠道权限进行效验、对入参进行效验
|
3 | 某某接口/未检查导出数量限制 | 对导出结果做效验 | 某某 | 轻微 | 对导出结果数量做限制 |
4 | 部分接口未做事务一致性效验 | 增加事务效验 | 某某 | 轻微 | 增加事务效验 |