From d9d0ce900265639811b526095e5243a658451a3e Mon Sep 17 00:00:00 2001 From: Cooper van Wijck Date: Mon, 2 Mar 2026 17:28:02 +1100 Subject: [PATCH] fix(net-lb-app-ext-regional): use list(string) for route_rules cors_policy attributes (#3776) * Update CORS policy in urlmap_config to use lists for allow_headers, allow_methods, and expose_headers * Update README.md to clarify urlmap_config structure and options for load balancer configuration --- modules/net-lb-app-ext-regional/README.md | 2 +- modules/net-lb-app-ext-regional/variables-urlmap.tf | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/modules/net-lb-app-ext-regional/README.md b/modules/net-lb-app-ext-regional/README.md index 77b6e96a9..2ced9173a 100644 --- a/modules/net-lb-app-ext-regional/README.md +++ b/modules/net-lb-app-ext-regional/README.md @@ -847,7 +847,7 @@ For deploying changes to load balancer configuration please refer to [net-lb-app | [ports](variables.tf#L189) | Optional ports for HTTP load balancer. | list(string) | | null | | [protocol](variables.tf#L204) | Protocol supported by this load balancer. | string | | "HTTP" | | [ssl_certificates](variables.tf#L222) | SSL target proxy certificates (only if protocol is HTTPS) for existing, custom, and managed certificates. | object({…}) | | {} | -| [urlmap_config](variables-urlmap.tf#L19) | The URL map configuration. | object({…}) | | {…} | +| [urlmap_config](variables-urlmap.tf#L19) | The URL map configuration. | object({…}) | | {…} | ## Outputs diff --git a/modules/net-lb-app-ext-regional/variables-urlmap.tf b/modules/net-lb-app-ext-regional/variables-urlmap.tf index 71318734e..90c67e617 100644 --- a/modules/net-lb-app-ext-regional/variables-urlmap.tf +++ b/modules/net-lb-app-ext-regional/variables-urlmap.tf @@ -230,12 +230,12 @@ variable "urlmap_config" { request_mirror_backend = optional(string) cors_policy = optional(object({ allow_credentials = optional(bool) - allow_headers = optional(string) - allow_methods = optional(string) + allow_headers = optional(list(string)) + allow_methods = optional(list(string)) allow_origin_regexes = list(string) allow_origins = list(string) disabled = optional(bool) - expose_headers = optional(string) + expose_headers = optional(list(string)) max_age = optional(string) })) fault_injection_policy = optional(object({