Merge pull request #16 from m-mizutani/fix/nil-error

Add handling for nil interface in clone function
m-mizutani · Feb 26, 2024 · 89175da · 89175da
2 parents ccdd4ed + a2fc6f4
commit 89175da
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/clone.go b/clone.go
@@ -107,6 +107,9 @@ func (x *masq) clone(fieldName string, src reflect.Value, tag string) reflect.Va
 		return dst
 
 	case reflect.Interface:
+		if src.IsNil() {
+			return src
+		}
 		return x.clone(fieldName, src.Elem(), tag)
 
 	default:

diff --git a/clone_test.go b/clone_test.go
@@ -316,3 +316,15 @@ func TestDirectUUID(t *testing.T) {
 
 	gt.S(t, buf.String()).Contains("stringer")
 }
+
+func TestNilInterface(t *testing.T) {
+	var buf bytes.Buffer
+	type myStruct struct {
+		Data interface{}
+	}
+	logger := slog.New(slog.NewJSONHandler(&buf, &slog.HandlerOptions{
+		ReplaceAttr: masq.New(),
+	}))
+	logger.Info("hello", slog.Any("test", myStruct{}))
+	gt.S(t, buf.String()).Contains("null")
+}