hyper-rustls Allow construction with HttpConnector and default ClientConfig

Closes: https://github.com/ctz/hyper-rustls/issues/67

Apr 01 '20 17:04 inikulin

Something is wrong with the CI for MacOS - builds get cancelled right away. It's definitely unrelated to the PR.

Apr 02 '20 12:04 inikulin

maybe a new_with_connector() is more suitable.

https://github.com/hyperium/hyper-tls/blob/fa6d6a71a5b2be91c827a35f9fafe5179b0ff4ca/src/client.rs#L71-L75

Aug 01 '20 16:08 kinosang

@kinosang thanks for the suggestion and we looked into it, but we'd like to stick with hyper-rustls in our project.

Aug 02 '20 14:08 inikulin

@inikulin yeah, I mean, to uniformize the APIs, we may have a new_with_connector() for hyper-rustls instead of a default_client_config.

Aug 02 '20 14:08 kinosang

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.

Aug 02 '20 14:08 inikulin

@inikulin please rebase

@lucab @ctz could you have a look at this PR? I need exactly the same thing and this looks like an easy change.

May 31 '21 07:05 ghost