From 1e9b3e9cdd2c7e5938df7bc925e008b1e6cb2935 Mon Sep 17 00:00:00 2001 From: Jens Reimann Date: Fri, 22 Nov 2024 11:35:02 +0100 Subject: [PATCH] fix: don't check if the bucket exists. Performing an "exists" check requires the s3:ListAllMyBuckets permission. Which feels excessive for such a simple check. Closes: https://github.com/trustification/trustify/issues/1038 --- modules/storage/src/service/s3.rs | 1 - 1 file changed, 1 deletion(-) diff --git a/modules/storage/src/service/s3.rs b/modules/storage/src/service/s3.rs index f133cd1d4..5e8d75842 100644 --- a/modules/storage/src/service/s3.rs +++ b/modules/storage/src/service/s3.rs @@ -35,7 +35,6 @@ impl S3Backend { CONTENT_ENCODING, HeaderValue::from_str(&compression.to_string())?, )]))?; - assert!(bucket.exists().await?, "S3 bucket not found"); log::info!( "Using S3 bucket '{}' in '{}' for doc storage", bucket.name,