Add s3 domain names with region so cloudfront origins outside of us-east-1 won't cause redirects immediately after setup

[kitsunde]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

Buckets created outside of us-east-1 will redirect to s3 when set as origins for cloudfront until DNS propagation has happened unless using the regional domain name. We need to set:

${aws_s3_bucket.default.bucket}.s3-${aws_s3_bucket.default.region}.amazonaws.com

It would be nice if we could get a regional_domain_name or maybe canonical_domain_name from the aws_s3_bucket resource, and even make it recommended for origin since it's rather surprising especially if the redirects get cached.

New or Affected Resource(s)`

• aws_s3_bucket

Potential Terraform Configuration

This

  origin {
    domain_name = "${aws_s3_bucket.default.domain_name}"
    origin_id = "s3-${var.environment}"
    s3_origin_config {
      origin_access_identity = "${aws_cloudfront_origin_access_identity.default.cloudfront_access_identity_path}"
    }
  }

Needs to be rewritten as:

  origin {
    domain_name = "${aws_s3_bucket.default.bucket}.s3-${aws_s3_bucket.default.region}.amazonaws.com"
    origin_id = "s3-${var.environment}"
    s3_origin_config {
      origin_access_identity = "${aws_cloudfront_origin_access_identity.default.cloudfront_access_identity_path}"
    }
  }

References

• https://forums.aws.amazon.com/thread.jspa?threadID=216814

[saravanan30erd]

PR submitted: #4556

[bflad]

The bucket_regional_domain_name attribute will be available in the aws_s3_bucket resource in version 1.21.0 of the AWS provider, releasing later today.

[bflad]

This has been released in version 1.21.0 of the AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!