Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add transferTrailer to switch whether forward trailer-relative header #13

Merged
merged 3 commits into from
Jul 4, 2023

Conversation

wzekin
Copy link
Contributor

@wzekin wzekin commented Jun 30, 2023

What type of PR is this?

feat

What this PR does / why we need it (English/Chinese):

En: In the reverse proxy, Trailer is a hop-by-hop Header, but there is often a need to transfer Trailer in the reverse generation, so a switch has been added to support the transfer of Trailer-related Header in the reverse proxy
Zh: 在反代中 Trailer 属于 hop-by-hop 的 Header,但经常有在反代中传输 Trailer 的需求,所以加入了一个开关,来支持在反代中可以传输 Trailer 相关的 Header

Which issue(s) this PR fixes:

@wzekin wzekin changed the title feat: do not delete 'Te: trailers' when forwarding request feat: add transferTrailer to switch whether forward trailer-relative header Jul 3, 2023
@wzekin wzekin force-pushed the feat/te_trailer branch from a7b3269 to 47700b6 Compare July 3, 2023 06:40
Copy link

@welkeyever welkeyever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@li-jin-gou li-jin-gou merged commit 331781c into hertz-contrib:master Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants