Review comment addressed

conductor-sdk · Feb 23, 2024 · 0e7e82a · 0e7e82a
1 parent a84be6c
commit 0e7e82a
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/Conductor/Client/ApiClient.cs b/Conductor/Client/ApiClient.cs
@@ -184,11 +184,9 @@ private RestResponse RetryRestClientCallApi(String path, Method method, List<Key
             RestResponse response = null;
             while (retryCount < Constants.MAX_TOKEN_REFRESH_RETRY_COUNT)
             {
-                bool isTokenRefreshed = false; // reset refreshStatusflag
-
                 var request = PrepareRequest(
-                    path, method, queryParams, postBody, headerParams, formParams, fileParams,
-                    pathParams, contentType);
+                path, method, queryParams, postBody, headerParams, formParams, fileParams,
+                pathParams, contentType);
 
                 InterceptRequest(request);
                 response = RestClient.Execute(request, method);
@@ -199,17 +197,17 @@ private RestResponse RetryRestClientCallApi(String path, Method method, List<Key
                 {
                     var JsonContent = JsonConvert.DeserializeObject<JObject>(response.Content);
 
-                    if (JsonContent["error"].ToString() == "EXPIRED_TOKEN" && retryCount < Constants.MAX_TOKEN_REFRESH_RETRY_COUNT)
+                    if (JsonContent["error"].ToString() == "EXPIRED_TOKEN")
                     {
                         string refreshToken = configuration.GetRefreshToken();
                         headerParams["X-Authorization"] = refreshToken;
-                        isTokenRefreshed = true;
                         retryCount++;
                     }
                 }
-
-                if (!isTokenRefreshed)
+                else
+                {
                     break;
+                }
             }
             return response;
         }