fix: reset current promise after every config request (#939)

DevCycleHQ · Aug 20, 2024 · 6710e1a · 6710e1a
1 parent 15a7718
commit 6710e1a
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/sdk/js/src/ConfigRequestConsolidator.ts b/sdk/js/src/ConfigRequestConsolidator.ts
@@ -89,8 +89,12 @@ export class ConfigRequestConsolidator {
             this.requestParams ? this.requestParams : undefined,
         )
         this.requestParams = null
-        const bucketedConfig = await this.currentPromise
-        this.currentPromise = null
+        const bucketedConfig = await this.currentPromise.finally(() => {
+            // clear the current promise so we can make another request
+            // this should happen regardless of whether the request was successful or not
+            this.currentPromise = null
+        })
+
         return bucketedConfig
     }
 }