patched; panic Event.Object() and Event.EmbedObject() with nil (#338)
This commit is contained in:
parent
0872592ea2
commit
c1533bd5f8
9
event.go
9
event.go
|
@ -207,6 +207,12 @@ func (e *Event) Object(key string, obj LogObjectMarshaler) *Event {
|
||||||
return e
|
return e
|
||||||
}
|
}
|
||||||
e.buf = enc.AppendKey(e.buf, key)
|
e.buf = enc.AppendKey(e.buf, key)
|
||||||
|
if obj == nil {
|
||||||
|
e.buf = enc.AppendNil(e.buf)
|
||||||
|
|
||||||
|
return e
|
||||||
|
}
|
||||||
|
|
||||||
e.appendObject(obj)
|
e.appendObject(obj)
|
||||||
return e
|
return e
|
||||||
}
|
}
|
||||||
|
@ -224,6 +230,9 @@ func (e *Event) EmbedObject(obj LogObjectMarshaler) *Event {
|
||||||
if e == nil {
|
if e == nil {
|
||||||
return e
|
return e
|
||||||
}
|
}
|
||||||
|
if obj == nil {
|
||||||
|
return e
|
||||||
|
}
|
||||||
obj.MarshalZerologObject(e)
|
obj.MarshalZerologObject(e)
|
||||||
return e
|
return e
|
||||||
}
|
}
|
||||||
|
|
|
@ -37,3 +37,29 @@ func TestEvent_AnErr(t *testing.T) {
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestEvent_ObjectWithNil(t *testing.T) {
|
||||||
|
var buf bytes.Buffer
|
||||||
|
e := newEvent(levelWriterAdapter{&buf}, DebugLevel)
|
||||||
|
_ = e.Object("obj", nil)
|
||||||
|
_ = e.write()
|
||||||
|
|
||||||
|
want := `{"obj":null}`
|
||||||
|
got := strings.TrimSpace(buf.String())
|
||||||
|
if got != want {
|
||||||
|
t.Errorf("Event.Object() = %q, want %q", got, want)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestEvent_EmbedObjectWithNil(t *testing.T) {
|
||||||
|
var buf bytes.Buffer
|
||||||
|
e := newEvent(levelWriterAdapter{&buf}, DebugLevel)
|
||||||
|
_ = e.EmbedObject(nil)
|
||||||
|
_ = e.write()
|
||||||
|
|
||||||
|
want := "{}"
|
||||||
|
got := strings.TrimSpace(buf.String())
|
||||||
|
if got != want {
|
||||||
|
t.Errorf("Event.EmbedObject() = %q, want %q", got, want)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
Loading…
Reference in New Issue