-
Notifications
You must be signed in to change notification settings - Fork 150
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
Allow construction with HttpConnector and default ClientConfig #107
base: main
Are you sure you want to change the base?
Conversation
fb04d58
to
f7bb45c
Compare
Something is wrong with the CI for MacOS - builds get cancelled right away. It's definitely unrelated to the PR. |
maybe a |
@kinosang thanks for the suggestion and we looked into it, but we'd like to stick with |
@inikulin yeah, I mean, to uniformize the APIs, we may have a |
This is what I did initially, however later we've had a case where we needed to do minor adjustments to the default config and this API is flexible enough to support that. |
Fix until rustls#107 is merged
Closes: #67