Fix bug introduced in #416 that would result in a breaking change if …

…an existing use has set diagnostic_settings with any of frontend_request_body_bytes, frontend_response_body_bytes, backend_request_body_bytes and/or backend_response_body_bytes not set
recognizegroup · Jul 12, 2024 · 4a5bae1 · 4a5bae1
1 parent ac8f24f
commit 4a5bae1
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/modules/azure/api_management/variables.tf b/modules/azure/api_management/variables.tf
@@ -77,10 +77,10 @@ variable "diagnostic_settings" {
     http_correlation_protocol    = string, # possible values: None, Legacy, W3C
     headers_to_log_request       = list(string),
     headers_to_log_response      = list(string),
-    frontend_request_body_bytes  = number,
-    frontend_response_body_bytes = number,
-    backend_request_body_bytes   = number,
-    backend_response_body_bytes  = number
+    frontend_request_body_bytes  = optional(number, 32),
+    frontend_response_body_bytes = optional(number, 32),
+    backend_request_body_bytes   = optional(number, 32),
+    backend_response_body_bytes  = optional(number, 32)
   })
   description = "Settings for api management diagnostic, api-management-diagnostic will be created only if api_management_logger_settings have been provided. "
   default = {