From 60014b1218f9830d6f891bf2adfe657f7381ac9b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tom=C3=A1=C5=A1=20Hozza?= Date: Mon, 7 Jul 2025 13:11:50 +0200 Subject: [PATCH] upload/koji: use passed transport in NewFromPlain() MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Don't create a new transport in NewFromPlain() using CreateRetryableTransport(), because the only place in which NewFromPlain() is called passes the return value of CreateRetryableTransport() when calling it. Signed-off-by: Tomáš Hozza --- internal/upload/koji/koji.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/internal/upload/koji/koji.go b/internal/upload/koji/koji.go index 653836b6d..656f10ae3 100644 --- a/internal/upload/koji/koji.go +++ b/internal/upload/koji/koji.go @@ -272,8 +272,7 @@ func NewFromPlain(server, user, password string, transport http.RoundTripper) (* // The API doesn't require sessionID, sessionKey and callnum yet, // so there's no need to use the custom Koji RoundTripper, // let's just use the one that the called passed in. - rhTransport := CreateRetryableTransport() - loginClient, err := xmlrpc.NewClient(server, rhTransport) + loginClient, err := xmlrpc.NewClient(server, transport) if err != nil { return nil, err }