Merge branch 'develop' of https://github.com/code4romania/teo into de…

…velop
code4romania · Dec 6, 2023 · 85fa6fb · 85fa6fb
2 parents edd1a26 + 19199e7
commit 85fa6fb
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 10 deletions.
diff --git a/terraform/cognito.tf b/terraform/cognito.tf
@@ -1,5 +1,4 @@
 # TODO:
-# - custom_domain
 # - sms_configuration
 
 resource "aws_cognito_user_pool" "pool" {
@@ -14,11 +13,15 @@ resource "aws_cognito_user_pool" "pool" {
     "phone_number",
   ]
 
+  auto_verified_attributes = [
+    "email",
+  ]
+
   username_configuration {
     case_sensitive = false
   }
 
-  mfa_configuration          = "ON"
+  mfa_configuration          = "OFF"
   sms_authentication_message = "Your verification code is {####}."
 
   verification_message_template {
@@ -28,12 +31,12 @@ resource "aws_cognito_user_pool" "pool" {
     sms_message          = "Your verification code is {####}. "
   }
 
-  software_token_mfa_configuration {
-    enabled = true
-  }
+  # software_token_mfa_configuration {
+  #   enabled = true
+  # }
 
   admin_create_user_config {
-    allow_admin_create_user_only = true
+    allow_admin_create_user_only = false
 
     invite_message_template {
       email_message = "Your username is {username} and temporary password is {####}. "

diff --git a/terraform/s3.tf b/terraform/s3.tf
@@ -4,10 +4,6 @@ module "s3_bucket_files" {
 
   bucket = "${local.namespace}-files"
 
-  control_object_ownership = true
-  object_ownership         = "BucketOwnerPreferred"
-  acl                      = "public-read"
-
   block_public_acls       = true
   block_public_policy     = true
   ignore_public_acls      = true