fix issue 5972

Signed-off-by: Lyndon-Li <lyonghui@vmware.com>
2023-03-24 14:38:26 +08:00 · 2023-03-24 14:38:26 +08:00 · 2dc8a920ca
parent 15d44724e7
commit 2dc8a920ca
2 changed files with 2 additions and 1 deletions
--- a/changelogs/unreleased/6028-Lyndon-Li
+++ b/changelogs/unreleased/6028-Lyndon-Li
@ -0,0 +1 @@
+Fix issue #5972, don't assume errorField as error type when dealing with logger.WithError
--- a/pkg/util/logging/log_counter_hook.go
+++ b/pkg/util/logging/log_counter_hook.go
@ -71,7 +71,7 @@ func (h *LogHook) Fire(entry *logrus.Entry) error {
 		entryMessage = fmt.Sprintf("%s name: /%s", entryMessage, nameField.(string))
 	}
 	if isErrorFieldPresent {
-		entryMessage = fmt.Sprintf("%s error: /%s", entryMessage, errorField.(error).Error())
+		entryMessage = fmt.Sprintf("%s error: /%v", entryMessage, errorField)
 	}

 	if isNamespacePresent {
				`@ -0,0 +1 @@`
				`Fix issue #5972, don't assume errorField as error type when dealing with logger.WithError`