fix: reuse TCP Connection (#1163)

Filirom1 · web-flow · commit 423f5e31c901 · 2025-03-28T13:52:07.000+01:00
Signed-off-by: Romain Philibert &lt;romain.philibert@worldline.com&gt;
diff --git a/keycloak/keycloak_client.go b/keycloak/keycloak_client.go
@@ -332,6 +332,7 @@ func (keycloakClient *KeycloakClient) sendRequest(ctx context.Context, request *
 	if err != nil {
 		return nil, "", fmt.Errorf("error sending request: %v", err)
 	}
+	defer response.Body.Close()
 
 	// Unauthorized: Token could have expired
 	// Forbidden: After creating a realm, following GETs for the realm return 403 until you refresh
@@ -354,10 +355,9 @@ func (keycloakClient *KeycloakClient) sendRequest(ctx context.Context, request *
 		if err != nil {
 			return nil, "", fmt.Errorf("error sending request after refresh: %v", err)
 		}
+		defer response.Body.Close()
 	}
 
-	defer response.Body.Close()
-
 	responseBody, err := io.ReadAll(response.Body)
 	if err != nil {
 		return nil, "", err

Original file line number	Diff line number	Diff line change
`@@ -332,6 +332,7 @@ func (keycloakClient KeycloakClient) sendRequest(ctx context.Context, request `
`332`	`332`	`if err != nil {`
`333`	`333`	`return nil, "", fmt.Errorf("error sending request: %v", err)`
`334`	`334`	`}`
	`335`	`+ defer response.Body.Close()`
`335`	`336`
`336`	`337`	`// Unauthorized: Token could have expired`
`337`	`338`	`// Forbidden: After creating a realm, following GETs for the realm return 403 until you refresh`
`@@ -354,10 +355,9 @@ func (keycloakClient KeycloakClient) sendRequest(ctx context.Context, request `
`354`	`355`	`if err != nil {`
`355`	`356`	`return nil, "", fmt.Errorf("error sending request after refresh: %v", err)`
`356`	`357`	`}`
	`358`	`+ defer response.Body.Close()`
`357`	`359`	`}`
`358`	`360`
`359`		`- defer response.Body.Close()`
`360`		`-`
`361`	`361`	`responseBody, err := io.ReadAll(response.Body)`
`362`	`362`	`if err != nil {`
`363`	`363`	`return nil, "", err`