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

Use oghttp2 HTTP/2 codec #29109

Open
alyssawilk opened this issue Aug 17, 2023 · 5 comments
Open

Use oghttp2 HTTP/2 codec #29109

alyssawilk opened this issue Aug 17, 2023 · 5 comments
Labels
area/http enhancement Feature requests. Not bugs or questions. no stalebot Disables stalebot from closing an issue tech debt

Comments

@alyssawilk
Copy link
Contributor

Parallel to #21245
Switching to oghttp2 allows

Theoretically no significant behavioral or performance differences (or we should turn them off =P)

@alyssawilk alyssawilk added enhancement Feature requests. Not bugs or questions. tech debt no stalebot Disables stalebot from closing an issue area/http labels Aug 17, 2023
@alyssawilk
Copy link
Contributor Author

cc @birenroy we have any internal docs for the last two points we can link here?

@milton0825
Copy link
Contributor

@alyssawilk what is the timeline for deprecating nghttp2?

@alyssawilk
Copy link
Contributor Author

Probably about a year (longer than the normal reloadable flag deprecation process, but it's also a larger change) but we can theoretically add an option to compile it out sooner if it's problematic.

@alyssawilk
Copy link
Contributor Author

Worth mentioning and unfortunately I forgot to link, #31086 switched the default to oghttp2 last quarter!

@milton0825
Copy link
Contributor

milton0825 commented Jan 8, 2024

We ran into issues with oghttp2 and I have created: #31710

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/http enhancement Feature requests. Not bugs or questions. no stalebot Disables stalebot from closing an issue tech debt
Projects
None yet
Development

No branches or pull requests

2 participants