Fix bug

Co-authored-by: Ezequiel Bellver <ebellver@itba.edu.ar>

README.md

@@ -66,6 +66,9 @@ Junto a cada función se especifica para qué se usa.
     + [modules/s3/main.tf](terraform/modules/s3/main.tf#L38)
     + [organization/cloudfront.tf](terraform/organization/cloudfront.tf#L13)
 
+- **trimsuffix**: remueve substring del final de un string.
+    + [modules/s3/main.tf](terraform/modules/s3/main.tf#L19)
+
 - **sha1**: computa el `SHA1` del string de la configuración del apigw para saber si se necesita forzar el redeploy del módulo.
     + [modules/apigw/main.tf](terraform/modules/apigw/main.tf#L103)
 

terraform/modules/s3/main.tf

@@ -8,7 +8,7 @@ resource "aws_s3_bucket" "this" {
 }
 
 resource "aws_s3_bucket_policy" "this" {
-  count = var.objects != {} ? 1 : 0
+  count = var.type == 1 ? 1 : 0
 
   bucket = aws_s3_bucket.this.id
   policy = data.aws_iam_policy_document.this.json

terraform/modules/s3/variables.tf

@@ -34,5 +34,5 @@ variable "bucket_acl" {
 variable "type" {
   type        = number
   default     = 1
-  description = "Determines the type of the bucket. 1 for static website. 2 for logs."
+  description = "Determines the type of the bucket. 1 for static website and 2 for logs."
 }

terraform/organization/locals.tf

@@ -1,5 +1,5 @@
 locals {
-  bucket_name = "bsmsapp-itba-cloud-computing-g3-test"
+  bucket_name = "bsmsapp"
   path        = "../resources"
 
   s3 = {