fix(S3): turned off verifySSL

ecmwf · Mar 4, 2024 · 27e8f8a · 27e8f8a
1 parent a682dfa
commit 27e8f8a
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/src/eckit/io/s3/aws/S3ClientAWS.cc b/src/eckit/io/s3/aws/S3ClientAWS.cc
@@ -84,15 +84,13 @@ void S3ClientAWS::configure(const S3Config& config) {
     configuration.disableImdsV1 = true;
 
     // setup region
-    // configuration.region = config.region;
     if (!config.region.empty()) { configuration.region = config.region; }
 
-    // configuration.scheme = Aws::Http::Scheme::HTTPS;
-    // configuration.verifySSL = false;
+    // configuration.scheme    = Aws::Http::Scheme::HTTPS;
+    configuration.verifySSL = false;
 
     // setup endpoint
-    /// @todo handle http/https possibly via scheme flag in S3Config
-    if (!config.endpoint.host().empty()) { configuration.endpointOverride = "http://" + config.endpoint.host(); }
+    if (!config.endpoint.host().empty()) { configuration.endpointOverride = config.endpoint.host(); }
     if (config.endpoint.port() > 0) { configuration.endpointOverride += ":" + std::to_string(config.endpoint.port()); }
 
     if (auto cred = S3Session::instance().getCredentials(config.endpoint)) {