fudiwei / DotNetCore.SKIT.FlurlHttpClient.ByteDance

可能是全网唯一的 C# 版字节跳动 SDK,封装全部已知的字节跳动 OpenAPI,包含字节小程序开放平台、抖音开放平台、抖店开放平台、巨量引擎开放平台、TikTok Developer API、TikTok Shop Open Platform 等模块,可跨平台,持续随官方更新。欢迎 Star/Fork/PR。QQ 交流群 875580418【满】、930461548【满】、611974621。
https://github.com/fudiwei/DotNetCore.SKIT.FlurlHttpClient
MIT License
241 stars 59 forks source link

fix #20 invalid sign #21

Closed code-jar closed 1 year ago

code-jar commented 1 year ago

修复方案为去除JsonIgnore特性标签,同时签名方法暴露,方便后续遇到签名问题时排查

fudiwei commented 1 year ago

感谢 PR 😄

单纯地把 GET 参数声明为 JSON 字段会破坏目前的自动化校验工具,而且也不符合 HTTP 语义(头条的有部分 API 是支持 GET 请求中携带请求体的)。

GenerateRequestSignature 方法增加第三个参数,由调用方传入一个 Dictionary<string, string> 显式声明哪些 GET 参数参与签名,似乎是更好的方式。

code-jar commented 1 year ago

感谢 PR 😄

单纯地把 GET 参数声明为 JSON 字段会破坏目前的自动化校验工具,而且也不符合 HTTP 语义(头条的有部分 API 是支持 GET 请求中携带请求体的)。

GenerateRequestSignature 方法增加第三个参数,由调用方传入一个 Dictionary<string, string> 显式声明哪些 GET 参数参与签名,似乎是更好的方式。

重新修改了下 你看看