refactoring the try block for null check

shubpal07 · shubpal07 · commit fdca7fe4eff2 · 2025-07-03T13:30:29.000Z
diff --git a/modules/management/kubectl-apply/main.tf b/modules/management/kubectl-apply/main.tf
@@ -29,7 +29,7 @@ locals {
   # 2. Identify URL-based manifests
   url_manifests = {
     for index, manifest in local.enabled_manifests : index => manifest
-    if startswith(try(manifest.source, ""), "http://") || startswith(try(manifest.source, ""), "https://")
+    if try(manifest.source, null) != null && (startswith(manifest.source, "http://") || startswith(manifest.source, "https://"))
   }
 
   # 3. Rebuild the map by populating the 'content' field for URLs based manifest

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ locals {`
`29`	`29`	`# 2. Identify URL-based manifests`
`30`	`30`	`url_manifests = {`
`31`	`31`	`for index, manifest in local.enabled_manifests : index => manifest`
`32`		`- if startswith(try(manifest.source, ""), "http://") \|\| startswith(try(manifest.source, ""), "https://")`
	`32`	`+ if try(manifest.source, null) != null && (startswith(manifest.source, "http://") \|\| startswith(manifest.source, "https://"))`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`# 3. Rebuild the map by populating the 'content' field for URLs based manifest`