gin-vue-admin icon indicating copy to clipboard operation
gin-vue-admin copied to clipboard

[Bug]: c.ShouldBind应处理错误,而不是忽略

Open myface-wang opened this issue 3 years ago • 0 comments

gin-vue-admin 版本

2.5.3

Node 版本

v16.16.0

Golang 版本

go 1.19.1

是否依旧存在

可以

bug描述

项目中所有接口参数转结构体均忽略错误,类似: _ = c.ShouldBindJSON(&user) 若转换产生错误,将导致所有传参丢失,从而导致查询数据库条件不可预测

例1,如何触发:结构体int类型,但传参NaN,(前端Number()转换产生错误) 例2,会发生的问题:若参数含有page,pagesize,解析错误后,其值也会丢失,导致sql查询不再分页,查询所有

修改建议

建议处理该错误

myface-wang avatar Sep 21 '22 06:09 myface-wang