Skip to content

Commit

Permalink
Enforce defaults preventing public S3 ACLs
Browse files Browse the repository at this point in the history
  • Loading branch information
johnf committed Oct 28, 2023
1 parent 80223f0 commit e18d6d4
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions cdk/lib/main-stack.ts
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,7 @@ export class MainStack extends cdk.Stack {
const metaBucket = new s3.Bucket(this, 'MetaBucket', {
bucketName: `${appName}-meta-${env}`,
encryption: s3.BucketEncryption.S3_MANAGED,
blockPublicAccess: s3.BlockPublicAccess.BLOCK_ALL,
enforceSSL: true,
removalPolicy: cdk.RemovalPolicy.RETAIN,
});
Expand All @@ -78,6 +79,7 @@ export class MainStack extends cdk.Stack {
this.catalogBucket = new s3.Bucket(this, 'CatalogBucket', {
bucketName: `${appName}-catalog-${env}`,
encryption: s3.BucketEncryption.S3_MANAGED,
blockPublicAccess: s3.BlockPublicAccess.BLOCK_ALL,
enforceSSL: true,
// TODO: Do we want tiering?
// intelligentTieringConfigurations: [ ],
Expand Down

0 comments on commit e18d6d4

Please sign in to comment.