tegg icon indicating copy to clipboard operation
tegg copied to clipboard

[Feature Request] 使用 egg-typebox-validate 作为默认的参数校验插件

Open fengmk2 opened this issue 3 years ago • 6 comments

是否可以引入 https://github.com/xiekw2010/egg-typebox-validate 成为默认的 validate 插件,之前的 egg-validate 在 ts 下其实不太满足。

cc @xiekw2010

fengmk2 avatar Dec 02 '21 13:12 fengmk2

@atian25 @killagu @whxaxes 怎么看?

fengmk2 avatar Dec 02 '21 13:12 fengmk2

示例 https://github.com/cnpm/cnpmcore/pull/12

fengmk2 avatar Dec 02 '21 15:12 fengmk2

image

可以考虑把 validate 直接作为一个装饰器,controller 里面专注业务代码,更干净了

xiekw2010 avatar Dec 03 '21 06:12 xiekw2010

这个加在入参上比较好,比如说同时有 query,params 的,每个参数都可以有教研。

killagu avatar Dec 03 '21 11:12 killagu

参数定义和校验一体化(包括lei)会比较好一点。

除了基础类型还有业务校验,这个根据 @xiekw2010 说的写法会比较好一些,不要侵入函数本身。

popomore avatar Jan 28 '22 03:01 popomore

@xiekw2010 继续推进?

fengmk2 avatar Jan 28 '22 15:01 fengmk2