feat(bucketLifecycle): allow for prefix

awslabs · Jul 20, 2023 · 893278d · 893278d
1 parent 0e75eeb
commit 893278d
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 5 deletions.
diff --git a/source/packages/@aws-accelerator/accelerator/lib/stacks/accelerator-stack.ts b/source/packages/@aws-accelerator/accelerator/lib/stacks/accelerator-stack.ts
@@ -530,6 +530,7 @@ export abstract class AcceleratorStack extends cdk.Stack {
         expiration: lifecycleRule.expiration,
         expiredObjectDeleteMarker: lifecycleRule.expiredObjectDeleteMarker,
         id: lifecycleRule.id,
+        prefix: lifecycleRule.prefix,
         noncurrentVersionExpiration: lifecycleRule.noncurrentVersionExpiration,
         noncurrentVersionTransitions,
         transitions,

diff --git a/source/packages/@aws-accelerator/accelerator/lib/stacks/logging-stack.ts b/source/packages/@aws-accelerator/accelerator/lib/stacks/logging-stack.ts
@@ -1273,7 +1273,7 @@ export class LoggingStack extends AcceleratorStack {
         kmsKey: this.cloudwatchKey,
         logRetentionInDays: this.props.globalConfig.cloudwatchLogRetentionInDays,
       };
-
+      
       this.centralLogsBucket = new CentralLogsBucket(this, 'CentralLogsBucket', {
         s3BucketName: this.centralLogsBucketName,
         serverAccessLogsBucket: serverAccessLogsBucket,

diff --git a/source/packages/@aws-accelerator/config/lib/common-types/types.ts b/source/packages/@aws-accelerator/config/lib/common-types/types.ts
@@ -338,6 +338,7 @@ export const lifecycleRuleConfig = t.interface({
   noncurrentVersionExpiration: optional(t.number),
   noncurrentVersionTransitions: optional(t.array(transition)),
   transitions: optional(t.array(transition)),
+  prefix: optional(t.string),
 });
 
 export const resourcePolicyStatement = t.interface({
@@ -355,6 +356,7 @@ export class LifeCycleRule implements t.TypeOf<typeof lifecycleRuleConfig> {
   readonly noncurrentVersionExpiration: number = 366;
   readonly noncurrentVersionTransitions: Transition[] = [];
   readonly transitions: Transition[] = [];
+  readonly prefix: string = '';
 }
 
 export const shareTargets = t.interface({

diff --git a/source/packages/@aws-accelerator/constructs/lib/aws-s3/bucket.ts b/source/packages/@aws-accelerator/constructs/lib/aws-s3/bucket.ts
@@ -59,6 +59,7 @@ export interface S3LifeCycleRule {
   noncurrentVersionExpiration: number;
   transitions: Transition[];
   noncurrentVersionTransitions: Transition[];
+  prefix: string;
 }
 
 /**
@@ -349,19 +350,20 @@ export class Bucket extends Construct {
           };
           noncurrentVersionTransitions.push(noncurrentVersionTransitionsConfig);
         }
-
-        this.lifecycleRules.push({
+        const lifecycle = {
           abortIncompleteMultipartUploadAfter: cdk.Duration.days(
             lifecycleRuleConfig.abortIncompleteMultipartUploadAfter,
           ),
           enabled: lifecycleRuleConfig.enabled,
           expiration: cdk.Duration.days(lifecycleRuleConfig.expiration),
+          prefix: lifecycleRuleConfig.prefix,
           transitions,
           noncurrentVersionTransitions,
           noncurrentVersionExpiration: cdk.Duration.days(lifecycleRuleConfig.noncurrentVersionExpiration),
           expiredObjectDeleteMarker: lifecycleRuleConfig.expiredObjectDeleteMarker,
-          id: `LifecycleRule${this.props.s3BucketName}`,
-        });
+          id: `LifecycleRule${lifecycleRuleConfig.prefix}${this.props.s3BucketName}`,
+        }
+        this.lifecycleRules.push(lifecycle);
       }
     } else {
       this.lifecycleRules.push({

diff --git a/source/packages/@aws-accelerator/constructs/test/aws-s3/bucket.test.ts b/source/packages/@aws-accelerator/constructs/test/aws-s3/bucket.test.ts
@@ -211,6 +211,7 @@ describe('Bucket', () => {
       s3LifeCycleRules: [
         {
           id: '1',
+          prefix: 'object-prefix',
           abortIncompleteMultipartUploadAfter: 1,
           enabled: true,
           expiration: 24,