Skip to content
This repository was archived by the owner on Oct 13, 2023. It is now read-only.

Commit d6cc247

Browse files
authored
Merge pull request #427 from thaJeztah/19.03_backport_40232-comply_with_gelf_spec
[19.03 backport] logger/gelf: Skip empty lines to comply with spec Upstream-commit: cf14fa7a233114525e8a21abe1e4fc153fae7d3a Component: engine
2 parents 403d9c3 + cdd97d5 commit d6cc247

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

  • components/engine/daemon/logger/gelf

components/engine/daemon/logger/gelf/gelf.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,10 @@ func newGELFUDPWriter(address string, info logger.Info) (gelf.Writer, error) {
166166
}
167167

168168
func (s *gelfLogger) Log(msg *logger.Message) error {
169+
if len(msg.Line) == 0 {
170+
return nil
171+
}
172+
169173
level := gelf.LOG_INFO
170174
if msg.Source == "stderr" {
171175
level = gelf.LOG_ERR

0 commit comments

Comments
 (0)