envoy icon indicating copy to clipboard operation
envoy copied to clipboard

Add the ability to turn off huffman for given http2 protocol option.

Open KBaichoo opened this issue 1 month ago • 5 comments

Commit Message: Add the ability to selectively remove huffman encoding headers as the sender for a given hop. Additional Description: This is just https://github.com/envoyproxy/envoy/pull/38270 without the wonked git history from being an old PR. Risk Level: low (off by default) Testing: Unit test, Running in Prod Docs Changes: n/a Release Notes: included Platform Specific Features: n/a Fixes: #38025

KBaichoo avatar Nov 26 '25 00:11 KBaichoo

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/). envoyproxy/api-shepherds assignee is @markdroth CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/). CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch). envoyproxy/dependency-shepherds assignee is @moderation

:cat:

Caused by: https://github.com/envoyproxy/envoy/pull/42263 was opened by KBaichoo.

see: more, trace.

Assigning @RyanTheOptimist as a reviewer (legacy from #38270). Feel free to reassign if needed. /assign @RyanTheOptimist

adisuissa avatar Nov 26 '25 13:11 adisuissa

Dependency change is the addition of a patch. Requires API review too

/lgtm deps

moderation avatar Nov 27 '25 17:11 moderation

/wait

RyanTheOptimist avatar Dec 03 '25 05:12 RyanTheOptimist

PTAL @markdroth

KBaichoo avatar Dec 08 '25 22:12 KBaichoo