🐛 [maps] Expand the Flatten function to support pointers

Author: acabarbaye

changes/20250811090744.bugfix

@@ -0,0 +1 @@
+:bug: [`maps`] Expand the `Flatten` function to support pointers

utils/maps/flatten.go

@@ -34,6 +34,9 @@ func Flatten(thing map[string]any) (result Map, err error) {
 
 func flatten(result map[string]string, prefix string, v reflect.Value) (err error) {
 	if v.Kind() == reflect.Interface {
+		if v.IsNil() {
+			return
+		}
 		v = v.Elem()
 	}
 	switch v.Kind() {
@@ -82,11 +85,16 @@ func flatten(result map[string]string, prefix string, v reflect.Value) (err erro
 		result[prefix] = v.String()
 	case reflect.Invalid:
 		result[prefix] = ""
+	case reflect.Ptr:
+		if v.IsNil() {
+			return
+		}
+		err = flatten(result, prefix, v.Elem())
 	default:
 		if v.IsZero() {
 			result[prefix] = ""
 		} else {
-			err = commonerrors.Newf(commonerrors.ErrUnknown, "unknown %v", v)
+			err = commonerrors.Newf(commonerrors.ErrUnknown, "unknown value '%v'", v)
 		}
 	}
 	return

utils/maps/flatten_test.go

@@ -2,6 +2,7 @@ package maps
 
 import (
 	"fmt"
+	"github.com/ARM-software/golang-utils/utils/field"
 	"testing"
 	"time"
 
@@ -27,6 +28,16 @@ func TestFlatten(t *testing.T) {
 				"bar": "baz",
 			},
 		},
+		{
+			Input: map[string]any{
+				"foo": "bar",
+				"bar": field.ToOptionalString("baz"),
+			},
+			Output: map[string]string{
+				"foo": "bar",
+				"bar": "baz",
+			},
+		},
 
 		{
 			Input: map[string]any{