Fix panic when annotation of ingress is empty

pinkavaj · pinkavaj · commit d2564874ab49 · 2023-05-29T11:27:28.000+02:00
When the annotation of ingress is not set, the returned value is nil
(not empty map). Trying to assign to this map leads to panic.

Signed-off-by: Jiří Pinkava &lt;j-pi@seznam.cz&gt;
diff --git a/pkg/router/util.go b/pkg/router/util.go
@@ -29,6 +29,9 @@ func includeLabelsByPrefix(labels map[string]string, includeLabelPrefixes []stri
 }
 
 func filterMetadata(meta map[string]string) map[string]string {
+	if meta == nil {
+		meta = map[string]string{}
+	}
 	// prevent Flux from overriding Flagger managed objects
 	meta[toolkitReconcileKey] = toolkitReconcileValue
 	meta[helmDriftDetectionKey] = toolkitReconcileValue

Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,9 @@ func includeLabelsByPrefix(labels map[string]string, includeLabelPrefixes []stri`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`func filterMetadata(meta map[string]string) map[string]string {`
	`32`	`+ if meta == nil {`
	`33`	`+ meta = map[string]string{}`
	`34`	`+ }`
`32`	`35`	`// prevent Flux from overriding Flagger managed objects`
`33`	`36`	`meta[toolkitReconcileKey] = toolkitReconcileValue`
`34`	`37`	`meta[helmDriftDetectionKey] = toolkitReconcileValue`