From 20c6fad7c22dd24fe052d92b3408b6f4010757b7 Mon Sep 17 00:00:00 2001
From: Diaa Sami <disami@redhat.com>
Date: Mon, 30 Jan 2023 12:16:14 +0100
Subject: [PATCH] osbuild-worker/koji: Add logging for koji requests/responses

---
 internal/upload/koji/koji.go | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/internal/upload/koji/koji.go b/internal/upload/koji/koji.go
index 084544e7a..ae2cbd9b2 100644
--- a/internal/upload/koji/koji.go
+++ b/internal/upload/koji/koji.go
@@ -478,13 +478,24 @@ func customCheckRetry(ctx context.Context, resp *http.Response, err error) (bool
 		}
 	}
 
+	if !shouldRetry {
+		logrus.Info("Not retrying: ", resp.Status)
+	}
+
 	return shouldRetry, retErr
 }
 
+func logUnsuccessfulResponses(logger rh.Logger, resp *http.Response) {
+	if !(resp.StatusCode >= 200 && resp.StatusCode < 300) {
+		logger.Printf("Unsuccessful request: ", resp)
+	}
+}
+
 func createCustomRetryableClient() *rh.Client {
 	client := rh.NewClient()
 	client.Logger = rh.LeveledLogger(&LeveledLogrus{logrus.StandardLogger()})
 	client.CheckRetry = customCheckRetry
+	client.ResponseLogHook = logUnsuccessfulResponses
 	return client
 }