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
Re-enable rustls #701
Re-enable rustls #701
Conversation
Is this PR waiting on something? Http2 support is missing entirely without this. |
I'd love to merge support for this, it's currently waiting on a |
# Conflicts: # src/connect.rs
Please, take a look here: https://github.com/glebpom/reqwest/tree/rustls. There are conflict resolutions and adaptation of this branch for the latest master branch. |
This is ready: https://crates.io/crates/hyper-rustls/0.19.0 💯 |
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.
🎉
This don't compile with the rustls feature enabled |
@nickelc This is fixed here https://github.com/EmbarkStudios/reqwest/pull/1/files |
Merge latest upstream master and implement additional structures according to updates in master
I think this PR should probably also uncomment the |
Thanks so much! I don't know why CI isn't trigger for this, I'll fiddle around. |
Merged in #747! |
I realized today that the latest version of reqwest only supports openssl.
I updated the dependencies and fixed a few build errors and the test cases are passing.