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(app): configure HTTP/2 server parameters #2924

Merged
merged 1 commit into from
Apr 24, 2024
Merged

Conversation

olix0r
Copy link
Member

@olix0r olix0r commented Apr 23, 2024

59ea792 expanded the proxy's HTTP/2 server parameters so that a variety of Hyper server settings may be controlled at runtime. There is currently no way for the control plane to influence these settings.

This commit adds environment-based configuration for the inbound and outbound proxy servers. All server parameters made configurable in 59ea792 are now configurable via environment variables. Variables take the form:

LINKERD2_PROXY_<PROXY>_SERVER_HTTP2_<PARAM>

Where <PROXY> is either INBOUND or OUTBOUND, and <PARAM> is the capitalized snake-case name of the parameter.

@olix0r olix0r requested a review from a team as a code owner April 23, 2024 23:11
59ea792 expanded the proxy's HTTP/2 server parameters so that a variety of
Hyper server settings may be controlled at runtime. There is currently no way
for the control plane to influence these settings.

This commit adds environment-based configuration for the inbound and outbound
proxy servers. All server parameters made configurable in 59ea792 are now
configurable via environment variables. Variables take the form:

    LINKERD2_PROXY_<PROXY>_SERVER_HTTP2_<PARAM>

Where `<PROXY>` is either `INBOUND` or `OUTBOUND`, and `<PARAM>` is the
capitalized snake-case name of the parameter.
@olix0r olix0r force-pushed the ver/h2-params-config branch from f07354f to 1d5b38f Compare April 23, 2024 23:27
@olix0r olix0r merged commit 15ac05b into main Apr 24, 2024
17 checks passed
@olix0r olix0r deleted the ver/h2-params-config branch April 24, 2024 20:56
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.

1 participant