From e3fad7bc0f9cbd2a4117594b3b70520ebfff1e52 Mon Sep 17 00:00:00 2001
From: Maria Florencia Caro <caromflorencia@gmail.com>
Date: Thu, 22 Jul 2021 13:27:42 +0100
Subject: [PATCH] Changing variable default value

---
 README.md    | 4 ++--
 variables.tf | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/README.md b/README.md
index 0e3da4e..d1ffe95 100644
--- a/README.md
+++ b/README.md
@@ -96,8 +96,8 @@ No modules.
 | <a name="input_website_cors_expose_headers"></a> [website\_cors\_expose\_headers](#input\_website\_cors\_expose\_headers) | (Optional) Specifies expose header in the response. | `list(string)` | `[]` | no |
 | <a name="input_website_cors_max_age_seconds"></a> [website\_cors\_max\_age\_seconds](#input\_website\_cors\_max\_age\_seconds) | (Optional) Specifies time in seconds that browser can cache the response for a preflight request. Defaults to 3600 | `number` | `3600` | no |
 | <a name="input_website_domain_name"></a> [website\_domain\_name](#input\_website\_domain\_name) | The domain name to use for the website | `string` | n/a | yes |
-| <a name="input_website_error_document"></a> [website\_error\_document](#input\_website\_error\_document) | (Optional) An absolute path to the document to return in case of a 4XX error. | `string` | `null` | no |
-| <a name="input_website_index_document"></a> [website\_index\_document](#input\_website\_index\_document) | Amazon S3 returns this index document when requests are made to the root domain or any of the subfolders. | `string` | `"index.html"` | no |
+| <a name="input_website_error_document"></a> [website\_error\_document](#input\_website\_error\_document) | (Optional) An absolute path to the document to return in case of a 4XX error. Defaults to 404.html | `string` | `"404.html"` | no |
+| <a name="input_website_index_document"></a> [website\_index\_document](#input\_website\_index\_document) | Amazon S3 returns this index document when requests are made to the root domain or any of the subfolders.  Defaults to index.html | `string` | `"index.html"` | no |
 | <a name="input_website_versioning_enabled"></a> [website\_versioning\_enabled](#input\_website\_versioning\_enabled) | (Optional) Enable versioning. Once you version-enable a bucket, it can never return to an unversioned state. You can, however, suspend versioning on that bucket. Defaults to false | `bool` | `false` | no |
 | <a name="input_website_versioning_mfa_delete"></a> [website\_versioning\_mfa\_delete](#input\_website\_versioning\_mfa\_delete) | (Optional) Enable MFA delete for either Change the versioning state of your bucket or Permanently delete an object version. Default is false. This cannot be used to toggle this setting but is available to allow managed buckets to reflect the state in AWS. | `bool` | `false` | no |
 | <a name="input_www_website_bucket_force_destroy"></a> [www\_website\_bucket\_force\_destroy](#input\_www\_website\_bucket\_force\_destroy) | (Optional, Default:false) A boolean that indicates all objects (including any locked objects) should be deleted from the bucket so that the bucket can be destroyed without error. These objects are not recoverable. | `bool` | `false` | no |
diff --git a/variables.tf b/variables.tf
index b4a5429..c239f24 100644
--- a/variables.tf
+++ b/variables.tf
@@ -26,15 +26,15 @@ variable "website_bucket_force_destroy" {
 }
 
 variable "website_index_document" {
-  description = "Amazon S3 returns this index document when requests are made to the root domain or any of the subfolders."
+  description = "Amazon S3 returns this index document when requests are made to the root domain or any of the subfolders.  Defaults to index.html"
   type        = string
   default     = "index.html"
 }
 
 variable "website_error_document" {
-  description = "(Optional) An absolute path to the document to return in case of a 4XX error."
+  description = "(Optional) An absolute path to the document to return in case of a 4XX error. Defaults to 404.html"
   type        = string
-  default     = null
+  default     = "404.html"
 }
 
 variable "website_cors_allowed_headers" {