diff --git a/README.md b/README.md index 05e99c9d..e6237afd 100644 --- a/README.md +++ b/README.md @@ -187,8 +187,9 @@ Available targets: | namespace | Namespace (e.g. `eg` or `cp`) | string | `` | no | | null | an empty string | string | `` | no | | origin_bucket | Origin S3 bucket name | string | `` | no | -| origin_force_destroy | Delete all objects from the bucket so that the bucket can be destroyed without error (e.g. `true` or `false`) | bool | `false` | no | +| origin_force_destroy | Delete all objects from the bucket so that the bucket can be destroyed without error (e.g. `true` or `false`) | bool | `false` | no | | origin_path | An optional element that causes CloudFront to request your content from a directory in your Amazon S3 bucket or your custom origin. It must begin with a /. Do not add a / at the end of the path. | string | `` | no | +| override_origin_bucket_policy | When using an existing origin bucket (through var.origin_bucket), setting this to 'false' will make it so the existing bucket policy will not be overriden | string | `true` | no | | parent_zone_id | ID of the hosted zone to contain this record (or specify `parent_zone_name`) | string | `` | no | | parent_zone_name | Name of the hosted zone to contain this record (or specify `parent_zone_id`) | string | `` | no | | price_class | Price class for this distribution: `PriceClass_All`, `PriceClass_200`, `PriceClass_100` | string | `PriceClass_100` | no | diff --git a/docs/terraform.md b/docs/terraform.md index a36d0c4b..ba2adc05 100644 --- a/docs/terraform.md +++ b/docs/terraform.md @@ -46,8 +46,9 @@ | namespace | Namespace (e.g. `eg` or `cp`) | string | `` | no | | null | an empty string | string | `` | no | | origin_bucket | Origin S3 bucket name | string | `` | no | -| origin_force_destroy | Delete all objects from the bucket so that the bucket can be destroyed without error (e.g. `true` or `false`) | bool | `false` | no | +| origin_force_destroy | Delete all objects from the bucket so that the bucket can be destroyed without error (e.g. `true` or `false`) | bool | `false` | no | | origin_path | An optional element that causes CloudFront to request your content from a directory in your Amazon S3 bucket or your custom origin. It must begin with a /. Do not add a / at the end of the path. | string | `` | no | +| override_origin_bucket_policy | When using an existing origin bucket (through var.origin_bucket), setting this to 'false' will make it so the existing bucket policy will not be overriden | string | `true` | no | | parent_zone_id | ID of the hosted zone to contain this record (or specify `parent_zone_name`) | string | `` | no | | parent_zone_name | Name of the hosted zone to contain this record (or specify `parent_zone_id`) | string | `` | no | | price_class | Price class for this distribution: `PriceClass_All`, `PriceClass_200`, `PriceClass_100` | string | `PriceClass_100` | no | diff --git a/main.tf b/main.tf index 5ea6521b..1e84782a 100644 --- a/main.tf +++ b/main.tf @@ -69,6 +69,7 @@ data "template_file" "default" { } resource "aws_s3_bucket_policy" "default" { + count = local.using_existing_origin && ! var.override_origin_bucket_policy ? 0 : 1 bucket = local.bucket policy = data.template_file.default.rendered } @@ -77,7 +78,7 @@ data "aws_region" "current" { } resource "aws_s3_bucket" "origin" { - count = signum(length(var.origin_bucket)) == 1 ? 0 : 1 + count = local.using_existing_origin ? 0 : 1 bucket = module.origin_label.id acl = "private" tags = module.origin_label.tags @@ -147,6 +148,8 @@ data "aws_s3_bucket" "selected" { } locals { + using_existing_origin = signum(length(var.origin_bucket)) == 1 + bucket = join( "", compact( diff --git a/variables.tf b/variables.tf index 4e9affe3..2fec68c6 100644 --- a/variables.tf +++ b/variables.tf @@ -81,6 +81,12 @@ variable "additional_bucket_policy" { description = "Additional policies for the bucket. If included in the policies, the variables `$${bucket_name}`, `$${origin_path}` and `$${cloudfront_origin_access_identity_iam_arn}` will be substituted. It is also possible to override the default policy statements by providing statements with `S3GetObjectForCloudFront` and `S3ListBucketForCloudFront` sid." } +variable "override_origin_bucket_policy" { + type = string + default = true + description = "When using an existing origin bucket (through var.origin_bucket), setting this to 'false' will make it so the existing bucket policy will not be overriden" +} + variable "origin_bucket" { type = string default = "" @@ -97,7 +103,7 @@ variable "origin_path" { variable "origin_force_destroy" { type = bool default = false - description = "Delete all objects from the bucket so that the bucket can be destroyed without error (e.g. `true` or `false`)" + description = "Delete all objects from the bucket so that the bucket can be destroyed without error (e.g. `true` or `false`)" } variable "bucket_domain_format" {