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

ability to access http/1.1 preserved header casing #2880

Closed
GlenDC opened this issue May 29, 2022 · 2 comments
Closed

ability to access http/1.1 preserved header casing #2880

GlenDC opened this issue May 29, 2022 · 2 comments
Labels
C-feature Category: feature. This is adding a new feature.

Comments

@GlenDC
Copy link
Contributor

GlenDC commented May 29, 2022

Is your feature request related to a problem? Please describe.

In #2574 it was clarified how the http/1.1 preserve casing feature works.
However I would like to have access to this private extension for purposes other than writing to a connection.

The order of headers already seems to be preserved when reading the headers of a request, which is awesome.
However it would be even better if I could also read the headers exactly as they were read, so including the casing.

Describe the solution you'd like

Be able to read header data as close to it being read from the wire as possible.
The header order already seems to be preserved, but the casing is not resected.
It might be possible already to preserve this for proxy reasons,
but it is not yet possible for .headers() reading out purposes.

Describe alternatives you've considered

The only other alternative I would see is log this data via an external tool like Wireshark,
but that brings in additional complexity, which I had hoped to solve with hyper directly.

@GlenDC GlenDC added the C-feature Category: feature. This is adding a new feature. label May 29, 2022
@seanmonstar
Copy link
Member

This is desired. The next step is to come up with a proposed public API. I'm tracking this in #2695, so I'll close here :)

@GlenDC
Copy link
Contributor Author

GlenDC commented May 31, 2022

Cool. Thanks a lot :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-feature Category: feature. This is adding a new feature.
Projects
None yet
Development

No branches or pull requests

2 participants