Re-introduce loop

eclipse-theia · Aug 12, 2024 · 8ac4717 · 8ac4717
1 parent e97839d
commit 8ac4717
Showing 1 changed file with 15 additions and 8 deletions.
diff --git a/dev-packages/ovsx-client/src/ovsx-http-client.ts b/dev-packages/ovsx-client/src/ovsx-http-client.ts
@@ -47,15 +47,22 @@ export class OVSXHttpClient implements OVSXClient {
     }
 
     protected async requestJson<R>(url: string): Promise<R> {
-        await this.rateLimiter.removeTokens(1);
-        const context = await this.requestService.request({
-            url,
-            headers: { 'Accept': 'application/json' }
-        });
-        if (context.res.statusCode === 429) {
-            console.warn('OVSX rate limit exceeded. Consider reducing the rate limit.');
+        const attempts = 5;
+        for (let i = 0; i < attempts; i++) {
+            await this.rateLimiter.removeTokens(i + 1);
+            const context = await this.requestService.request({
+                url,
+                headers: { 'Accept': 'application/json' }
+            });
+            if (context.res.statusCode === 429) {
+                console.warn('OVSX rate limit exceeded. Will perform request with higher timeout. Consider reducing the rate limit.');
+                if (i < attempts - 1) {
+                    continue;
+                }
+            }
+            return RequestContext.asJson<R>(context);
         }
-        return RequestContext.asJson<R>(context);
+        throw new Error('Request timed out due to too many requests');
     }
 
     protected buildUrl(url: string, query?: object): string {