Regression: 0.48 doesn't allow connecting via HTTP at all #1980
Comments
|
Also encountered this issue. Would definitely like an option to continue using plain HTTP in a testing context. Release notes should've mentioned this change if this was intentional. |
ditto, also encountered this issue. |
|
Same problem here - for internal use only on an already-secure, wired-only network, I would like to continue using cleartext HTTP - can #1981 be merged? |
|
Please dear god fix this so I can connect to localstack containers without generating certs. |
|
You can still allow http by using fn s3_client(local_port: u16) -> S3Client {
let hyper_builder = Client::builder();
let http_connector = hyper_rustls::HttpsConnectorBuilder::new()
.with_native_roots()
.https_or_http()
.enable_http1()
.build();
let s3_http_client = HttpClient::from_builder(hyper_builder, http_connector);
let credentials = DefaultCredentialsProvider::new().unwrap();
S3Client::new_with(
s3_http_client,
credentials,
Region::Custom {
name: "Localstack".to_string(),
endpoint: format!("http://localhost:{}", local_port),
},
)
} |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Context: Our internal testing setup uses a minio instance so we can test against an actual S3 API, verify that signing URLs works as expected etc.
df31792 did not only update hyper-rustls and expose an option to use native or webpki roots, it also hardcoded the configuration to only support HTTPS without offering any way to configure this:
At least in some cases (like the one I outlined above), I think using
.https_or_http()instead would be useful, so I would like to see it either made configurable (at runtime or compile-time), or just as the default to not break my testing setup right now (though that's a bit selfish lol.)The text was updated successfully, but these errors were encountered: