durch / rust-s3

Rust library for interfacing with S3 API compatible services
MIT License
498 stars 195 forks source link

Feature request: No verify SSL should be a parameter rather than a feature #379

Open mdecimus opened 3 months ago

mdecimus commented 3 months ago

Hi,

I would like to propose specifying no_verify_ssl as a function parameter rather than a crate feature:

https://github.com/durch/rust-s3/blob/5105a1bff57827f88fb90840f03193665ea34d68/s3/src/request/tokio_backend.rs#L31

We interact with different object stores, some of them have valid certificates while others have self-signed certificates. Currently we need to disable TLS verification for all hosts in order to be able to support both self-signed and valid certificates.