only configure lifecycle policy if at least one expiration class has …

…been configured
stv0g · Jun 10, 2022 · 49e0188 · 49e0188
1 parent 02ac7f9
commit 49e0188
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/pkg/server/setup.go b/pkg/server/setup.go
@@ -79,13 +79,15 @@ func (s *Server) Setup() error {
 		})
 	}
 
-	if _, err := s.PutBucketLifecycleConfiguration(&s3.PutBucketLifecycleConfigurationInput{
-		Bucket: aws.String(s.Config.Bucket),
-		LifecycleConfiguration: &s3.BucketLifecycleConfiguration{
-			Rules: lcRules,
-		},
-	}); err != nil {
-		return fmt.Errorf("failed to set bucket %s's lifecycle rules: %w", s.Config.Bucket, err)
+	if len(lcRules) > 0 {
+		if _, err := s.PutBucketLifecycleConfiguration(&s3.PutBucketLifecycleConfigurationInput{
+			Bucket: aws.String(s.Config.Bucket),
+			LifecycleConfiguration: &s3.BucketLifecycleConfiguration{
+				Rules: lcRules,
+			},
+		}); err != nil {
+			return fmt.Errorf("failed to set bucket %s's lifecycle rules: %w", s.Config.Bucket, err)
+		}
 	}
 
 	// lc, err := svc.GetBucketLifecycleConfiguration(&s3.GetBucketLifecycleConfigurationInput{