search

bytedance / go-tagexpr

An interesting go struct tag expression syntax for field validation, etc.
Apache License 2.0
1.66k stars 142 forks source link

fix: unsafe gjson #75

Closed FGYFFFF closed 10 months ago

FGYFFFF commented 11 months ago

自带的 gjson unmarshal 用了 unsafe 操作。在一些场景下用户传来的 body 会被修改，导致绑定后的数据的内存也被一同改变，出现数据不一致的问题。

这个提出来讨论下，要怎么规避下这个问题，毕竟这个 copy 一次开销还是很大的。 不过标准的 json 库为了保证安全，这里总是要 copy 一份的

dugenkui03 commented 11 months ago

单元测试 没过

FGYFFFF commented 11 months ago

单元测试 没过

这个有办法重跑单测吗？ 单测挂的位置和我改动的位置上应该无任何关联啊