-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Closed
hyperium/hyper-util
#111Labels
A-serverArea: server.Area: server.C-featureCategory: feature. This is adding a new feature.Category: feature. This is adding a new feature.E-mediumEffort: medium. Some knowledge of how hyper internal works would be useful.Effort: medium. Some knowledge of how hyper internal works would be useful.K-hyper-utilCrate: hyper-utilCrate: hyper-util
Description
Is your feature request related to a problem? Please describe.
The current conn::auto::Connection future always internally determines the appropriate protocol version to use. However, this decision should have already been made by ALPN when running over TLS (or via prior knowledge), so the ReadVersion logic ends up adding some overhead and allows bad clients to use an HTTP version they didn't negotiate.
Describe the solution you'd like
I'd like a version of Connection that's essentially just an enum of the h1 and h2 Connection futures and has to be explicitly told which to use, like the http2_only and http1_only methods in 0.14.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-serverArea: server.Area: server.C-featureCategory: feature. This is adding a new feature.Category: feature. This is adding a new feature.E-mediumEffort: medium. Some knowledge of how hyper internal works would be useful.Effort: medium. Some knowledge of how hyper internal works would be useful.K-hyper-utilCrate: hyper-utilCrate: hyper-util