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

Fix rustls-webpki feature #1979

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

joshtriplett
Copy link
Contributor

@joshtriplett joshtriplett commented Apr 26, 2022

#1965 modified the initialization
code for HttpClient in a way that prevents it from building with the
rustls-webpki feature enabled. Fix that, so that building with
rustls-webpki works again.

Also, add a rustls-webpki forwarding feature for generated crates.

Would it be possible to get an 0.48.1 release with this change included? #1651 mentions 0.48 being the last release of rusoto; I'd really appreciate having this feature working in the final release.

Thank you so much to everyone who has worked on Rusoto!

rusoto#1965 modified the initialization
code for HttpClient in a way that prevents it from building with the
`rustls-webpki` feature enabled. Fix that, so that building with
rustls-webpki works again.
@iliana
Copy link
Contributor

iliana commented Apr 26, 2022

Is it possible to avoid the forwarding feature? The real maintenance cost of Rusoto is the several hours it takes for a script to build and publish the hundreds of service crates, and if I can get away with pushing only a new 0.48.1 for core, I’d like to.

I assume you can reference rusoto_core directly in your Cargo.toml and it’ll do the right thing.

@iliana
Copy link
Contributor

iliana commented Apr 26, 2022

Ah, you already dropped that commit in the time it took me to write that here :)

@joshtriplett
Copy link
Contributor Author

joshtriplett commented May 13, 2022

Following up on this: would it be possible to get a rusoto_core release with this fix? I'd love to drop my last use of openssl.

Not time-sensitive, just wanted to check in and see if there's anything I could do to help.

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.

2 participants