Improve error handling

dgreif · Nov 5, 2023 · c007ee6 · c007ee6
1 parent f518f37
commit c007ee6
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/packages/ring-client-api/rest-client.ts b/packages/ring-client-api/rest-client.ts
@@ -60,8 +60,9 @@ export interface ExtendedResponse {
 
 async function requestWithRetry<T>(
   requestOptions: RequestOptions & { url: string; allowNoResponse?: boolean },
-  retryCount: number,
+  retryCount?: number,
 ): Promise<T & ExtendedResponse> {
+  retryCount = retryCount || 0
   try {
     const options = {
         ...defaultRequestOptions,
@@ -83,7 +84,7 @@ async function requestWithRetry<T>(
     }
     return data
   } catch (e: any) {
-    if (!e.response && !requestOptions.allowNoResponse && retryCount >= 3) {
+    if (!e.response && !requestOptions.allowNoResponse && retryCount % 3) {
       logError(
         `Failed to reach Ring server at ${requestOptions.url}.  ${e.message}.  Trying again in 5 seconds...`,
       )
@@ -95,7 +96,7 @@ async function requestWithRetry<T>(
       logDebug(e)
 
       await delay(5000)
-      return requestWithRetry(requestOptions, retryCount ? 1 : retryCount++)
+      return requestWithRetry(requestOptions, retryCount++)
     }
     throw e
   }
@@ -486,10 +487,7 @@ export class RingRestClient {
         logError(`Request to ${url} failed:`)
         logError(e)
       }
-    }
-
-    if (!options.allowNoResponse) {
-      throw e
+      return response
     }
   }