Skip to content

Commit b3546c7

Browse files
authored
Merge pull request #227 from tablexi/fix-s3-encryption
adding default encryption to s3 buckets for cloudtrail
2 parents bca53ae + 8150e03 commit b3546c7

1 file changed

Lines changed: 18 additions & 0 deletions

File tree

aws/cloudtrail/main.tf

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,15 @@ resource "aws_s3_bucket" "logs" {
1616
acl = "log-delivery-write"
1717

1818
tags = var.tags
19+
20+
server_side_encryption_configuration {
21+
rule {
22+
bucket_key_enabled = false
23+
apply_server_side_encryption_by_default {
24+
sse_algorithm = "AES256"
25+
}
26+
}
27+
}
1928
}
2029

2130
resource "aws_s3_bucket" "mod" {
@@ -29,6 +38,15 @@ resource "aws_s3_bucket" "mod" {
2938
target_bucket = aws_s3_bucket.logs.id
3039
target_prefix = var.name
3140
}
41+
42+
server_side_encryption_configuration {
43+
rule {
44+
bucket_key_enabled = false
45+
apply_server_side_encryption_by_default {
46+
sse_algorithm = "AES256"
47+
}
48+
}
49+
}
3250
}
3351

3452
data "aws_iam_policy_document" "s3" {

0 commit comments

Comments
 (0)