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

build: add support for rustls #69

Merged
merged 1 commit into from
Nov 22, 2022

Conversation

seanpianka
Copy link
Contributor

@seanpianka seanpianka commented Nov 22, 2022

isahc pulls in native-tls by default. For users of httpmock that cannot use openssl, they need to activate a variation of the rustls feature for isahc to avoid seg-faults.

This PR adds support for rustls with webpki (or native-certs), but keeps the default features of httpmock with native-tls.

Blocked by sagebind/isahc#199

Signed-off-by: Sean Pianka [email protected]

isahc pulls in native-tls by default. For users of httpmock that cannot
use openssl, they need to activate a variation of the `rustls` feature
for isahc to avoid seg-faults.

This PR adds support for rustls with webpki (or native-certs), but keeps
the default features of httpmock with native-tls.

Signed-off-by: Sean Pianka <[email protected]>
@alexliesenfeld
Copy link
Owner

alexliesenfeld commented Nov 22, 2022

Thank you so much!

It seems sagebind/isahc#199 is still not stabilized yet, so we will need to wait until this PR can be shipped.

@alexliesenfeld alexliesenfeld merged commit 718212c into alexliesenfeld:master Nov 22, 2022
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