fix: remove Promise.finally

alibaba · Dec 20, 2021 · 7a76ae3 · 7a76ae3
1 parent f985ed7
commit 7a76ae3
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/packages/hooks/src/useRequest/src/utils/cachePromise.ts b/packages/hooks/src/useRequest/src/utils/cachePromise.ts
@@ -10,9 +10,16 @@ const setCachePromise = (cacheKey: CachedKey, promise: Promise<any>) => {
   // Because the promise.finally will change the reference of the promise
   cachePromise.set(cacheKey, promise);
 
-  promise.finally(() => {
-    cachePromise.delete(cacheKey);
-  });
+  // no use promise.finally for compatibility
+  promise
+    .then((res) => {
+      cachePromise.delete(cacheKey);
+      return res;
+    })
+    .catch((err) => {
+      cachePromise.delete(cacheKey);
+      throw err;
+    });
 };
 
 export { getCachePromise, setCachePromise };