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

Upgrade to tokio 1.0 #18

Merged
merged 3 commits into from
Feb 26, 2021
Merged

Upgrade to tokio 1.0 #18

merged 3 commits into from
Feb 26, 2021

Conversation

e00E
Copy link
Contributor

@e00E e00E commented Dec 24, 2020

This should not yet be merged as some dependencies have not yet had a new release with tokio 1.0 support as seen by git dependencies.
Ready to merge.

I changed the doc example in lib.rs to the hyper example https://github.com/hyperium/hyper/blob/master/examples/client.rs because I got annoyed at finding the right combinators.

@dizda
Copy link

dizda commented Jan 2, 2021

👍

@FelipeRosa FelipeRosa mentioned this pull request Jan 6, 2021
@FelipeRosa
Copy link

Hey folks,

Just giving a heads up that these dependencies were updated to use tokio 1.0:

tokio-native-tls = "0.3.0"
tokio-rustls = "0.22.0"
hyper-tls = "0.5.0"

@e00E
Copy link
Contributor Author

e00E commented Jan 6, 2021

Thanks for the update Felipe. There are no longer any git dependencies. The PR is ready to be merged.

@nagua
Copy link
Contributor

nagua commented Jan 16, 2021

It seems like typed_headers is pulling in bytes 0.5 and tokio 1.0 changed to using bytes 1.0.
As some crates in the ecosystem needed to upgrade to bytes 1 to be usable with tokio 1.0 I think
we need to do that too...

I made the changes in a branch on my fork, if you think that it is a good idea, feel free to add the commit to your PR.
https://github.com/nagua/hyper-proxy/tree/upgrade-tokio

@nagua
Copy link
Contributor

nagua commented Jan 16, 2021

I just found out, that clippy isn't fond of wildcard dependencies (and rightly so ;) ). So I changed the dev-dependencies back to actual versions.

@@ -2,19 +2,20 @@
//!
//! # Example
//! ```rust,no_run
//! use hyper::{Client, Request, Uri};
//! use hyper::{Client, Request, Uri, body::HttpBody};
//! use hyper::client::HttpConnector;
//! use futures::{TryFutureExt, TryStreamExt};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These types are no longer required. Also, the README.md example should be updated.

massand added a commit to massand/hyper-proxy-1 that referenced this pull request Feb 18, 2021
The hyper-openssl support is added as a new feature to hyper-proxy. It is used to
perform crypto operations using keypairs via OpenSSL engine. This commit also adds
support for Tokio 1.0 (adapted from tafia/hyper-proxy#18),
needed to interoperate with native-tls and hyper crates that are already upgraded
to Tokio 1.0.
@ghost
Copy link

ghost commented Feb 25, 2021

@tafia can you please merge and publish to crates.io?

@tafia tafia merged commit 8919228 into tafia:master Feb 26, 2021
@tafia
Copy link
Owner

tafia commented Feb 26, 2021

Sorry for that!
I'll deploy tonight

@ghost
Copy link

ghost commented Feb 26, 2021

Thank you, much appreciated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants