Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: ACL value can be null even when using Terragrunt #75

Merged
merged 2 commits into from
Mar 22, 2021

Conversation

antonbabenko
Copy link
Member

@antonbabenko antonbabenko commented Mar 22, 2021

References:
gruntwork-io/terragrunt#892

terragrunt.hcl:

inputs = {
  bucket        = "my-awesome-logs"

  acl    = "null"   # <= It is not possible to pass `null` here from terragrunt, so passing `"null"` as string instead.
  grant = [
    {
    type        = "CanonicalUser"
    permissions = ["FULL_CONTROL"]
    id          = dependency.aws-data.outputs.aws_canonical_user_id
    }, {
    type        = "CanonicalUser"
    permissions = ["FULL_CONTROL"]
    id          = "c4c1ede66af53448b93c283ce9448c4ba468c9432aa01d700d3878632f77d2d0"
    # Ref. https://github.com/terraform-providers/terraform-provider-aws/issues/12512
    # Ref. https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/AccessLogs.html
  }]
}

@antonbabenko antonbabenko merged commit 4072c22 into master Mar 22, 2021
@antonbabenko antonbabenko deleted the fix-acl-null branch March 22, 2021 10:15
@antonbabenko
Copy link
Member Author

v1.22.0 has been just released.

@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 29, 2022
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant