From 1065c3dc447789948eaef50f63f260ddda9b6c6e Mon Sep 17 00:00:00 2001 From: Francis Guimond Date: Wed, 17 May 2023 14:16:40 -0400 Subject: [PATCH] Log handler error at error level instead of info level (#5011) Signed-off-by: Francis Guimond --- CHANGELOG-6.md | 1 + backend/pipeline/handler/legacy.go | 6 +++++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/CHANGELOG-6.md b/CHANGELOG-6.md index 2eac66588d..6e4569471b 100644 --- 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 index 79e4976295..aef06ba748 100644 --- 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 {