-
-
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
feat(client): add 1.0 compatible client conn API #3155
Conversation
51b339f
to
99b76f7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey! I went over and compared this with master. I know these won't the same everywhere, but I think what I have highlighted should be the same.
struct TokioExec; | ||
impl<F> hyper::rt::Executor<F> for TokioExec | ||
where | ||
F: std::future::Future + Send + 'static, | ||
F::Output: Send + 'static, | ||
{ | ||
fn execute(&self, fut: F) { | ||
tokio::spawn(fut); | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: should we move this outside the test (but inside the backports test module)?
fc45467
to
3d41ee1
Compare
This patch backports client/conn/http1 and http2 modules from 1.0 to ease transition. It allows code still using 0.14.x to switch to the per-version Connection types available in 1.0.
3d41ee1
to
3e08efc
Compare
This patch backports client/conn/http1 and http2 modules from 1.0 to ease transition. It allows code still using 0.14.x to switch to the per-version Connection types available in 1.0. Closes hyperium#3053 Co-authored-by: KOVACS Tamas <[email protected]>
This patch backports client/conn/http1 and http2 modules from 1.0 to ease transition. It allows code still using 0.14.x to switch to the per-version Connection types available in 1.0.
Updating (part of) #3063