fix: Handle empty interface when marshalling (liip#26)

Co-authored-by: Canh Nguyen <[email protected]>

sheriff.go

@@ -52,6 +52,9 @@ type Marshaller interface {
 // In all other cases we can't derive the type in a meaningful way and is therefore an `interface{}`.
 func Marshal(options *Options, data interface{}) (interface{}, error) {
 	v := reflect.ValueOf(data)
+	if !v.IsValid() {
+		return data, nil
+	}
 	t := v.Type()
 
 	// Initialise nestedGroupsMap,

sheriff_test.go

@@ -663,3 +663,15 @@ func TestMarshal_AliaString(t *testing.T) {
 	_, err := Marshal(&Options{}, &v)
 	assert.NoError(t, err)
 }
+
+type EmptyInterfaceStruct struct {
+	Data interface{} `json:"data"`
+}
+
+func TestMarshal_EmptyInterface(t *testing.T) {
+	v := EmptyInterfaceStruct{}
+	o := &Options{}
+
+	_, err := Marshal(o, v)
+	assert.NoError(t, err)
+}