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

support h2 for http proxy client connector #438

Open
GlenDC opened this issue Mar 3, 2025 · 0 comments
Open

support h2 for http proxy client connector #438

GlenDC opened this issue Mar 3, 2025 · 0 comments
Labels
bug Something isn't working easy An easy issue to pick up for anyone. good first issue Good for newcomers mentor available A mentor is available to help you through the issue.
Milestone

Comments

@GlenDC
Copy link
Member

GlenDC commented Mar 3, 2025

the current client http proxy client connector is hardcoded for h1. This is fine and dandy for most use cases, but in case you are dealing with a h2 only server this won't work.

https://github.com/plabayo/rama/blob/0abc1495acf91bd6a1eda72bf1662eee424a9db3/rama-http-backend/src/client/proxy/layer/proxy_connector/connector.rs

Goal would be to make it auto by default. but that one can also set it to h1 or h2.
The auto mode would try to use the request protocol version and falling back to h1 if it couldn't be extracted.

@GlenDC GlenDC added bug Something isn't working easy An easy issue to pick up for anyone. good first issue Good for newcomers labels Mar 3, 2025
@GlenDC GlenDC added this to the v0.2 milestone Mar 3, 2025
@GlenDC GlenDC added the mentor available A mentor is available to help you through the issue. label Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working easy An easy issue to pick up for anyone. good first issue Good for newcomers mentor available A mentor is available to help you through the issue.
Projects
None yet
Development

No branches or pull requests

1 participant