Skip to content

Commit

Permalink
Do not reference non-existent object member
Browse files Browse the repository at this point in the history
  • Loading branch information
Nuru committed Feb 22, 2022
1 parent 550e49b commit b3babb4
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ resource "aws_s3_bucket" "default" {
#bridgecrew:skip=BC_AWS_S3_16:Skipping `Ensure S3 bucket versioning is enabled` because dynamic blocks are not supported by checkov
#bridgecrew:skip=BC_AWS_S3_14:Skipping `Ensure all data stored in the S3 bucket is securely encrypted at rest` because variables are not understood
#bridgecrew:skip=BC_AWS_GENERAL_56:Skipping `Ensure that S3 buckets are encrypted with KMS by default` because we do not have good defaults
#bridgecrew:skip=BC_AWS_GENERAL_72:We do not agree that cross-region replication must be enabled
count = local.enabled ? 1 : 0
bucket = local.bucket_name
force_destroy = var.force_destroy
Expand Down Expand Up @@ -61,7 +62,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "default" {

content {
id = rule.value.id
status = try(rule.value.enabled == true ? "Enabled" : "Disabled", rule.value.status)
status = rule.value.enabled == true ? "Enabled" : "Disabled"

# Filter is always required due to https://github.com/hashicorp/terraform-provider-aws/issues/23299
filter {
Expand Down

0 comments on commit b3babb4

Please sign in to comment.