From f541d398144ab03403d5031642b490df20ec8d72 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Peter=20Scha=CC=88fer?= <101886095+PeterSchafer@users.noreply.github.com> Date: Sat, 8 Feb 2025 17:45:52 +0100 Subject: [PATCH] fix: Add missing error nil check to log msg --- internal/analysis/analysis.go | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/internal/analysis/analysis.go b/internal/analysis/analysis.go index 4d3107e..ac685c7 100644 --- a/internal/analysis/analysis.go +++ b/internal/analysis/analysis.go @@ -509,7 +509,9 @@ func (a *analysisOrchestrator) RunTest(ctx context.Context, orgId string, b bund parsedResponse, err := testApi.ParseCreateTestResponse(resp) defer func() { closeErr := resp.Body.Close() - a.logger.Err(closeErr).Msg("failed to close response body") + if closeErr != nil { + a.logger.Err(closeErr).Msg("failed to close response body") + } }() if err != nil { a.logger.Debug().Msg(err.Error()) @@ -574,7 +576,9 @@ func (a *analysisOrchestrator) retrieveTestURL(ctx context.Context, client *test } defer func() { closeErr := httpResponse.Body.Close() - a.logger.Err(closeErr).Msg("failed to close response body") + if closeErr != nil { + a.logger.Err(closeErr).Msg("failed to close response body") + } }() parsedResponse, err := testApi.ParseGetTestResultResponse(httpResponse)