Skip to content

Commit aed920d

Browse files
committed
Allow using multiple cors origins
1 parent 8334f18 commit aed920d

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

main.tf

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -354,15 +354,14 @@ resource "aws_s3_bucket_cors_configuration" "origin" {
354354

355355
bucket = one(aws_s3_bucket.origin).id
356356

357-
dynamic "cors_rule" {
358-
for_each = distinct(compact(concat(var.cors_allowed_origins, var.aliases, var.external_aliases)))
359-
content {
360-
allowed_headers = var.cors_allowed_headers
361-
allowed_methods = var.cors_allowed_methods
362-
allowed_origins = [cors_rule.value]
363-
expose_headers = var.cors_expose_headers
364-
max_age_seconds = var.cors_max_age_seconds
365-
}
357+
cors_rule {
358+
allowed_headers = var.cors_allowed_headers
359+
allowed_methods = var.cors_allowed_methods
360+
allowed_origins = sort(
361+
distinct(compact(concat(var.cors_allowed_origins, var.aliases, var.external_aliases)))
362+
)
363+
expose_headers = var.cors_expose_headers
364+
max_age_seconds = var.cors_max_age_seconds
366365
}
367366

368367
depends_on = [time_sleep.wait_for_aws_s3_bucket_settings]

0 commit comments

Comments
 (0)