From 96b42b57919440b798b2171f45792efcf1c148f1 Mon Sep 17 00:00:00 2001 From: Sanne Raymaekers Date: Wed, 8 Feb 2023 16:33:13 +0100 Subject: [PATCH] internal/worker: add String to clienterrors.Error struct This should make the logging better when errors have another error struct. Fixes #3272 --- internal/worker/clienterrors/errors.go | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/internal/worker/clienterrors/errors.go b/internal/worker/clienterrors/errors.go index 1d35da15f..14da197ff 100644 --- a/internal/worker/clienterrors/errors.go +++ b/internal/worker/clienterrors/errors.go @@ -1,5 +1,9 @@ package clienterrors +import ( + "fmt" +) + const ( ErrorNoDynamicArgs ClientErrorCode = 1 ErrorInvalidTargetConfig ClientErrorCode = 2 @@ -46,6 +50,10 @@ type Error struct { Details interface{} `json:"details,omitempty"` } +func (e *Error) String() string { + return fmt.Sprintf("Code: %d, Reason: %s, Details: %v", e.ID, e.Reason, e.Details) +} + const ( JobStatusSuccess = "2xx" JobStatusUserInputError = "4xx"