fix: root object via variable to avoid creation everytime (#12)

Author: kaustubhposhiya

EXAMPLE.md

@@ -4,7 +4,7 @@ Below is an examples of calling this module.
 ## Cloudfront distribution with S3 origin
 ```
 module "cloudfront" {
-  source = "./cloudfront"
+  source = "git::https://github.com/TechHoldingLLC/terraform-aws-cloudfront.git?ref=<TAG>"
   origin = [
     {
       domain_name = "s3_bucket_regional_domain_name"
@@ -28,7 +28,7 @@ module "cloudfront" {
 ## Cloudfront distribution with custom http and https origin endpoint
 ```
 module "cloudfront" {
-  source     = "./cloudfront"
+  source     = "git::https://github.com/TechHoldingLLC/terraform-aws-cloudfront.git?ref=<TAG>"
   origin = [
     {
       domain_name = "s3_bucket_regional_domain_name"
@@ -74,7 +74,7 @@ module "cloudfront" {
 ## Cloudfront distribution with s3 Origin with TTL value
 ```
 module "cloudfront" {
-  source = "./cloudfront"
+  source = "git::https://github.com/TechHoldingLLC/terraform-aws-cloudfront.git?ref=<TAG>"
   origin = [
     {
       domain_name = "s3_bucket_regional_domain_name"
@@ -106,7 +106,7 @@ module "cloudfront" {
 ## Cloudfront distribution with multiple origin and cache behavior
 ```
 module "cloudfront" {
-  source = "./cloudfront"
+  source = "git::https://github.com/TechHoldingLLC/terraform-aws-cloudfront.git?ref=<TAG>"
   origin = [
     {
       domain_name = "domain_name"

README.md

@@ -9,7 +9,6 @@
 | Name | Version |
 |------|---------|
 | <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.0 |
-| <a name="provider_local"></a> [local](#provider\_local) | n/a |
 
 ## Modules
 
@@ -22,7 +21,6 @@ No modules.
 | [aws_cloudfront_distribution.cloudfront](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/cloudfront_distribution) | resource |
 | [aws_route53_record.cloudfront](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_record) | resource |
 | [aws_s3_object.object](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_object) | resource |
-| [local_file.default_object](https://registry.terraform.io/providers/hashicorp/local/latest/docs/resources/file) | resource |
 
 ## Inputs
 
@@ -38,6 +36,7 @@ No modules.
 | <a name="input_default_cache_behaviour_target_origin_id"></a> [default\_cache\_behaviour\_target\_origin\_id](#input\_default\_cache\_behaviour\_target\_origin\_id) | Target origin ID for default cache behaviour | `string` | n/a | yes |
 | <a name="input_default_cache_forwarded_values"></a> [default\_cache\_forwarded\_values](#input\_default\_cache\_forwarded\_values) | forwarded values for default cache behavior | `any` | `{}` | no |
 | <a name="input_default_root_object"></a> [default\_root\_object](#input\_default\_root\_object) | Default root object | `string` | `"index.html"` | no |
+| <a name="input_default_root_object_content"></a> [default\_root\_object\_content](#input\_default\_root\_object\_content) | Content for the default root object | `string` | `""` | no |
 | <a name="input_domain_aliases"></a> [domain\_aliases](#input\_domain\_aliases) | domain aliases | `list(string)` | `null` | no |
 | <a name="input_enabled"></a> [enabled](#input\_enabled) | Cloudfront state | `bool` | `true` | no |
 | <a name="input_function_association"></a> [function\_association](#input\_function\_association) | Function association | `list(any)` | `[]` | no |

s3.tf

@@ -3,20 +3,10 @@ resource "aws_s3_object" "object" {
 
   bucket       = var.s3_bucket_name
   key          = "index.html"
-  source       = "${path.cwd}/${path.module}/s3/index.html"
+  content      = var.default_root_object_content
   content_type = "text/html"
 
   # The filemd5() function is available in Terraform 0.11.12 and later
   # For Terraform 0.11.11 and earlier, use the md5() function and the file() function:
-  # etag = "${md5(file("path/to/file"))}"
-  # etag = filemd5("${path.cwd}/${path.module}/s3/index.html")
-
-  depends_on = [ local_file.default_object ]
+  etag = md5(var.default_root_object_content)
 }
-
-resource "local_file" "default_object" {
-  count = var.put_default_root_object ? 1 : 0
-
-  content  = ""
-  filename = "${path.cwd}/${path.module}/s3/index.html"
-}

variables.tf

@@ -145,6 +145,12 @@ variable "put_default_root_object" {
   default     = false
 }
 
+variable "default_root_object_content" {
+  description = "Content for the default root object"
+  type        = string
+  default     = ""
+}
+
 variable "s3_bucket_name" {
   description = "s3 bucket name for flag"
   type        = string