upload/koji: use passed transport in NewFromPlain()

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 <thozza@redhat.com>
This commit is contained in:
Tomáš Hozza 2025-07-07 13:11:50 +02:00 committed by Tomáš Hozza
parent 55744ac1e8
commit 60014b1218

View file

@ -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
}