Refactored AblyHttpClient code as per review comments

ably · Apr 19, 2024 · df35dbd · df35dbd
1 parent 9a80c7e
commit df35dbd
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/IO.Ably.Shared/Http/AblyHttpClient.cs b/src/IO.Ably.Shared/Http/AblyHttpClient.cs
@@ -88,7 +88,7 @@ public async Task<AblyResponse> Execute(AblyRequest request)
             int currentTry = 0;
             var startTime = Now();
 
-            var maxNumberOfRetries = Options.HttpMaxRetryCount; // One for the first request
+            var maxNumberOfRetries = Options.HttpMaxRetryCount;
             var host = GetHost();
 
             request.Headers.TryGetValue("request_id", out var requestId);
@@ -148,7 +148,7 @@ public async Task<AblyResponse> Execute(AblyRequest request)
                     throw new AblyException(new ErrorInfo(WrapWithRequestId("Error executing request. " + ex.Message), ErrorCodes.InternalError), ex);
                 }
             }
-            while (currentTry <= maxNumberOfRetries);
+            while (currentTry <= maxNumberOfRetries); // 1 primary host and remaining fallback hosts
 
             throw new AblyException(new ErrorInfo(WrapWithRequestId("Error executing request, exceeded max no. of retries"), ErrorCodes.InternalError));