From 190ff83caab67d97ce0b644d41c25271ce845e7f Mon Sep 17 00:00:00 2001 From: Florian Wilhelm Date: Wed, 6 May 2020 16:27:34 +0200 Subject: [PATCH] Handle case when stepname is not set for logging (#1517) --- pkg/log/log.go | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/pkg/log/log.go b/pkg/log/log.go index 74254dbe4..05d911f48 100644 --- a/pkg/log/log.go +++ b/pkg/log/log.go @@ -24,11 +24,16 @@ const ( func (formatter *PiperLogFormatter) Format(entry *logrus.Entry) (bytes []byte, err error) { message := "" + stepName := entry.Data["stepName"] + if stepName == nil { + stepName = "(noStepName)" + } + switch formatter.logFormat { case logFormatDefault: - message = fmt.Sprintf("%-5s %-6s - %s\n", entry.Level, entry.Data["stepName"], entry.Message) + message = fmt.Sprintf("%-5s %-6s - %s\n", entry.Level, stepName, entry.Message) case logFormatWithTimestamp: - message = fmt.Sprintf("%s %-5s %-6s - %s\n", entry.Time.Format("15:04:05"), entry.Level, entry.Data["stepName"], entry.Message) + message = fmt.Sprintf("%s %-5s %-6s - %s\n", entry.Time.Format("15:04:05"), entry.Level, stepName, entry.Message) case logFormatPlain: message = entry.Message + "\n" default: