[scrubber] skip if interface is nil (#20418)

* [scrubber] skip if interface is nil

* update

Author: iQQBot

components/scrubber/scrubber.go

@@ -388,6 +388,9 @@ func (s *scrubberImpl) deepCopyStruct(fieldName string, src reflect.Value, scrub
 		return dst
 
 	case reflect.Interface:
+		if src.IsNil() {
+			return src
+		}
 		dst := reflect.New(src.Elem().Type())
 		copied := s.deepCopyStruct(fieldName, src.Elem(), scrubTag, skipScrub)
 		dst.Elem().Set(copied)

components/scrubber/scrubber_test.go

@@ -444,6 +444,42 @@ func TestDeepCopyStruct(t *testing.T) {
 			},
 			CmpOpts: []cmp.Option{cmpopts.IgnoreUnexported(UnexportedStructToTest{})},
 		},
+		{
+			Name: "nil interface",
+			Struct: &struct {
+				Hashed       string `scrub:"hash"`
+				NilInterface interface{}
+			}{
+				Hashed: "foo",
+			},
+			Expectation: Expectation{
+				Result: &struct {
+					Hashed       string `scrub:"hash"`
+					NilInterface interface{}
+				}{
+					Hashed:       "[redacted:md5:acbd18db4cc2f85cedef654fccc4a4d8]",
+					NilInterface: nil,
+				},
+			},
+		},
+		{
+			Name: "nil point interface",
+			Struct: &struct {
+				Hashed       string `scrub:"hash"`
+				NilInterface *string
+			}{
+				Hashed: "foo",
+			},
+			Expectation: Expectation{
+				Result: &struct {
+					Hashed       string `scrub:"hash"`
+					NilInterface *string
+				}{
+					Hashed:       "[redacted:md5:acbd18db4cc2f85cedef654fccc4a4d8]",
+					NilInterface: nil,
+				},
+			},
+		},
 	}
 
 	for _, test := range tests {