Merge pull request #5 from druids/feat/fargate-compatibility

feat/fargate

Author: trebidav

container.tf

@@ -11,14 +11,14 @@ module "container_definition_noalb" { // Without ALB
   container_memory_reservation = var.memory
   container_memory             = var.memory_limit
 
-  port_mappings = [
+  port_mappings = var.port ? [
     {
       containerPort = var.port
-      hostPort      = 0
+      hostPort      = contains(var.requires_compatibilities, "FARGATE") ? var.port : 0
       protocol      = "tcp"
     },
-  ]
-
+  ] : []
+ 
   log_configuration = {
     logDriver = "awslogs"
     options = {

variables.tf

@@ -35,7 +35,7 @@ variable "private_subnet_ids" {
 
 variable "port" {
   description = "port on which the service listens"
-  default     = 80
+  default     = null
   type        = number
 }