cloudwego / hertz

Go HTTP framework with high-performance and strong-extensibility for building micro-services.
https://www.cloudwego.io
Apache License 2.0
5.04k stars 491 forks source link

fix(hz): client header and path tag #1118

Closed FGYFFFF closed 2 months ago

FGYFFFF commented 3 months ago

What type of PR is this?

fix

Check the PR title.

(Optional) Translate the PR title into Chinese.

hz client 生成的 path/header参数不默认做 snake style

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en: client header and path tag zh(optional): path/header参数不默认做 snake style

image

(Optional) Which issue(s) this PR fixes:

(Optional) The PR that updates user documentation:

welkeyever commented 2 months ago

这个变更背景是啥,path/header不再受 snake_tag flag 控制了?

FGYFFFF commented 2 months ago

这个变更背景是啥,path/header不再受 snake_tag flag 控制了?

是的,path/header 的生成本身就不受 snake_flag 控制;只是在 client 解析的时候把他们多解析了一遍,导致预期外的错误