updated comments

modules/aws/transfer_family/main.tf

@@ -158,11 +158,6 @@ resource "aws_transfer_user" "this" {
   tags                = var.tags
   user_name           = each.value.username
 
-  home_directory_mappings {
-    entry  = "/"
-    target = "/${module.bucket.s3_bucket_id}/$${Transfer:UserName}"
-  }
-
   dynamic "home_directory_mappings" {
     # Disables the dynamic block of home_directory_mappings if home_directory_type is not "LOGICAL"
     for_each = each.value.home_directory_type == "LOGICAL" ? [1] : []

modules/aws/transfer_family/variables.tf

@@ -184,9 +184,9 @@ variable "vpc_id" {
 variable "users" {
   description = "(Optional) A map of user names and their configuration"
   type = map(object({
-    home_directory      = oprional(string)
+    home_directory      = oprional(string)            # Cannot be set if home_directory_type is set to "LOGICAL".
     home_directory_type = optional(string, "LOGICAL") # Default is "LOGICAL"
-    policy              = optional(string)
+    policy              = optional(string)            # Set for a custom session policy see https://docs.aws.amazon.com/transfer/latest/userguide/requirements-roles.html#session-policy for more information
     username            = string
   }))
   default = {}