-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Split the server Connection type into version-specific submodules #2851
Comments
Hey @seanmonstar, I think I'd like to work on this next. However, I need a few additional pointers before I can start:
I'd ask you for clarifications, confirmation that we still want to work on this, and maybe some new thoughts that you had about the topic. |
Yes, this is something that still needs to be done.
Lemme take a look... I think that the types currently in
Some duplication is fine. It's also fine to refactor to some free functions in
Those should likely be removed, to start with. We can add them back in a way that |
Do the connections then become simple-wrappers around the underlying dispatchers?
|
The
server::conn
type,Connection
, should get version-specific types in submoduleshttp1
andhttp2
.The text was updated successfully, but these errors were encountered: