Create new http request on each retry (#125)

pkg/token/httpclient/httpclient_test.go

@@ -126,7 +126,7 @@ func setTestCase() ([]testCaseIssuer, []testCaseIdentity) {
 				name: "no context",
 				url:  "https://hpe-greenlake-tenant.okta.com/oauth2/default",
 				ctx:  nil,
-				err:  errors.New("net/http: nil Context"),
+				err:  errors.New("network error in post to get token"),
 			},
 			{
 				name:       "status code 400",

pkg/token/issuertoken/issuertoken.go

@@ -40,15 +40,16 @@ func GenerateToken(
 		return "", err
 	}
 
-	// Create the request
-	req, err := http.NewRequestWithContext(ctx, http.MethodPost, clientURL, strings.NewReader(params.Encode()))
-	if err != nil {
-		return "", err
-	}
-	req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
-
 	// Execute the request, with retries
 	resp, err := tokenutil.DoRetries(func() (*http.Response, error) {
+		// Create the request
+		req, errReq := createRequest(ctx, params, clientURL)
+		if errReq != nil {
+			return nil, errReq
+		}
+		// Close the request after use, i.e. don't reuse the TCP connection
+		req.Close = true
+
 		return httpClient.Do(req)
 	}, retryLimit)
 	if err != nil {
@@ -76,6 +77,17 @@ func GenerateToken(
 	return token.AccessToken, nil
 }
 
+// createRequest creates a new http request
+func createRequest(ctx context.Context, params url.Values, clientURL string) (*http.Request, error) {
+	req, err := http.NewRequestWithContext(ctx, http.MethodPost, clientURL, strings.NewReader(params.Encode()))
+	if err != nil {
+		return nil, err
+	}
+	req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
+
+	return req, nil
+}
+
 // generateParamsAndURL generates the parameters and URL for the request
 func generateParamsAndURL(clientID, clientSecret, identityServiceURL, iamVersion string) (url.Values, string, error) {
 	params := url.Values{}

pkg/token/token-util/token-util.go

@@ -89,6 +89,7 @@ func DoRetries(call func() (*http.Response, error), retries int) (*http.Response
 			break
 		}
 
+		log.Printf("Retrying request, retries left: %v", retries)
 		time.Sleep(3 * time.Second)
 		retries--
 	}