Merge pull request #1 from DNXLabs/terraform-0.12-support

Adds terraform 0.12 support

_data.tf

@@ -1,3 +1,3 @@
 data "aws_s3_bucket" "selected" {
-  bucket = "${var.s3_bucket_id}"
+  bucket = var.s3_bucket_id
 }

_variables.tf

@@ -3,7 +3,7 @@ variable "name" {}
 variable "s3_bucket_id" {}
 
 variable "hostnames" {
-    type = "list"
+  type = list(string)
 }
 
 variable "hosted_zone" {}

cloudfront.tf

@@ -5,27 +5,27 @@ resource "aws_cloudfront_origin_access_identity" "default" {
 resource "aws_cloudfront_distribution" "default" {
   enabled             = true
   is_ipv6_enabled     = true
-  comment             = "${var.name}"
-  aliases             = "${var.hostnames}"
+  comment             = var.name
+  aliases             = var.hostnames
   price_class         = "PriceClass_All"
   default_root_object = "index.html"
 
   origin {
-    domain_name = "${data.aws_s3_bucket.selected.bucket_regional_domain_name}"
+    domain_name = data.aws_s3_bucket.selected.bucket_regional_domain_name
     origin_id   = "s3Origin"
 
     s3_origin_config {
-      origin_access_identity = "${aws_cloudfront_origin_access_identity.default.cloudfront_access_identity_path}"
+      origin_access_identity = aws_cloudfront_origin_access_identity.default.cloudfront_access_identity_path
     }
   }
 
   dynamic "logging_config" {
-    for_each = "${compact([var.cloudfront_logging_bucket])}"
+    for_each = compact([var.cloudfront_logging_bucket])
 
     content {
       include_cookies = false
-      bucket          = "${var.cloudfront_logging_bucket}"
-      prefix          = "${var.cloudfront_logging_prefix}"
+      bucket          = var.cloudfront_logging_bucket
+      prefix          = var.cloudfront_logging_prefix
     }
   }
 
@@ -50,7 +50,7 @@ resource "aws_cloudfront_distribution" "default" {
   }
 
   viewer_certificate {
-    acm_certificate_arn      = "${var.certificate_arn}"
+    acm_certificate_arn      = var.certificate_arn
     ssl_support_method       = "sni-only"
     minimum_protocol_version = "TLSv1.1_2016"
   }

route53.tf

@@ -1,13 +1,13 @@
 data "aws_route53_zone" "selected" {
-  name    = "${var.hosted_zone}"
+  name = var.hosted_zone
 }
 
 resource "aws_route53_record" "hostname" {
-  count   = "${var.hostname_create ? length(var.hostnames) : 0}"
+  count = var.hostname_create ? length(var.hostnames) : 0
 
-  zone_id = "${data.aws_route53_zone.selected.zone_id}"
-  name    = "${var.hostnames[count.index]}"
+  zone_id = data.aws_route53_zone.selected.zone_id
+  name    = var.hostnames[count.index]
   type    = "CNAME"
   ttl     = "300"
-  records = ["${aws_cloudfront_distribution.default.domain_name}"]
+  records = [aws_cloudfront_distribution.default.domain_name]
 }

s3-policy.tf

@@ -5,22 +5,22 @@ data "aws_iam_policy_document" "s3_policy" {
 
     principals {
       type        = "AWS"
-      identifiers = ["${aws_cloudfront_origin_access_identity.default.iam_arn}"]
+      identifiers = [aws_cloudfront_origin_access_identity.default.iam_arn]
     }
   }
 
   statement {
     actions   = ["s3:ListBucket"]
-    resources = ["${data.aws_s3_bucket.selected.arn}"]
+    resources = [data.aws_s3_bucket.selected.arn]
 
     principals {
       type        = "AWS"
-      identifiers = ["${aws_cloudfront_origin_access_identity.default.iam_arn}"]
+      identifiers = [aws_cloudfront_origin_access_identity.default.iam_arn]
     }
   }
 }
 
 resource "aws_s3_bucket_policy" "oai" {
-  bucket = "${var.s3_bucket_id}"
-  policy = "${data.aws_iam_policy_document.s3_policy.json}"
+  bucket = var.s3_bucket_id
+  policy = data.aws_iam_policy_document.s3_policy.json
 }