Log handler error at error level instead of info level (#5011)

Signed-off-by: Francis Guimond <francis@sensu.io>
sensu · May 17, 2023 · 1065c3d · 1065c3d
1 parent cedb7a3
commit 1065c3d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/CHANGELOG-6.md b/CHANGELOG-6.md
@@ -15,6 +15,7 @@ Versioning](http://semver.org/spec/v2.0.0.html).
 - Added a session store, used to detect and prevent refresh token reuse
 
 ### Changed
+- Log handler error at error level instead of info level
 - Users are now automatically logged out after a period of inactivity (12h)
 
 ## [6.9.2] - 2023-03-08

diff --git a/backend/pipeline/handler/legacy.go b/backend/pipeline/handler/legacy.go
@@ -84,7 +84,11 @@ func (l *LegacyAdapter) Handle(ctx context.Context, ref *corev2.ResourceReferenc
 		}
 		fields["status"] = result.Status
 		fields["output"] = result.Output
-		logger.WithFields(fields).Info("event pipe handler executed")
+		if result.Status == 0 {
+			logger.WithFields(fields).Info("event pipe handler executed")
+		} else {
+			logger.WithFields(fields).Error("event pipe handler returned non ok status code")
+		}
 	case "tcp", "udp":
 		err := l.socketHandler(ctx, handler, event, mutatedData)
 		if err != nil {