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

[bp/1.29] http: keep trailers TE header instead of removing it (#32255) #33018

Merged
merged 1 commit into from
Mar 21, 2024

Commits on Mar 20, 2024

  1. http: keep trailers TE header instead of removing it (envoyproxy#32255

    )
    
    Official grpc library does not allow empty TE header. This causes an issue when proxying requests through Envoy which currently deletes the TE header since envoyproxy#30535.
    
    I opened a related MR on the grpc library to allow this header to be empty as the HTTP/2 RFC allows it but it might cause issues with other libraries.
    
    Risk Level: medium (behavioral change)
    Testing: modified an existing test
    Docs Changes: n/a
    Release Notes: inline
    Platform Specific Features:
    Runtime guard: keep the previous one
    
    Signed-off-by: Nathanael DEMACON <ndemacon@scaleway.com>
    Signed-off-by: Ryan Northey <ryan@synca.io>
    quantumsheep authored and phlax committed Mar 20, 2024
    Configuration menu
    Copy the full SHA
    212b2db View commit details
    Browse the repository at this point in the history