@cloudbase/manager-node 3.12.0 → 3.12.1-beta

package/lib/storage/index.js

@@ -894,26 +894,40 @@ class StorageService {
     async uploadFilesWithRetry({ uploadFiles, options, times, interval, failedFiles }) {
         const { files, onFileFinish } = options;
         const tempFailedFiles = [];
+        let curError = null;
         const res = await uploadFiles(Object.assign(Object.assign({}, options), { files: failedFiles.length
                 ? files.filter(file => failedFiles.includes(file.Key))
                 : files, onFileFinish: (...args) => {
                 const error = args[0];
                 const fileInfo = args[2];
                 if (error) {
+                    curError = error;
                     tempFailedFiles.push(fileInfo.Key);
                 }
                 onFileFinish === null || onFileFinish === void 0 ? void 0 : onFileFinish.apply(null, args);
             } }));
-        if (!(tempFailedFiles === null || tempFailedFiles === void 0 ? void 0 : tempFailedFiles.length) || times <= 0)
+        // if (!tempFailedFiles?.length || times <= 0) return res
+        if (!(tempFailedFiles === null || tempFailedFiles === void 0 ? void 0 : tempFailedFiles.length)) {
             return res;
-        if (times > 0) {
-            setTimeout(() => this.uploadFilesWithRetry({
-                uploadFiles,
-                options,
-                times: times - 1,
-                interval,
-                failedFiles: tempFailedFiles
-            }), interval);
+        }
+        else {
+            if (times > 0) {
+                return await new Promise((resolve, reject) => {
+                    setTimeout(() => this.uploadFilesWithRetry({
+                        uploadFiles,
+                        options,
+                        times: times - 1,
+                        interval,
+                        failedFiles: tempFailedFiles
+                    }).then(res => resolve(res))
+                        .catch(err => reject(err)), interval);
+                });
+            }
+            else {
+                if (curError) {
+                    throw curError;
+                }
+            }
         }
     }
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@cloudbase/manager-node",
-    "version": "3.12.0",
+    "version": "3.12.1-beta",
     "description": "The node manage service api for cloudbase.",
     "main": "lib/index.js",
     "scripts": {

package/src/storage/index.ts

@@ -1198,6 +1198,7 @@ export class StorageService {
     private async uploadFilesWithRetry({ uploadFiles, options, times, interval, failedFiles }) {
         const { files, onFileFinish } = options
         const tempFailedFiles = []
+        let curError: any = null
         const res = await uploadFiles({
             ...options,
             files: failedFiles.length
@@ -1207,24 +1208,37 @@ export class StorageService {
                 const error = args[0]
                 const fileInfo = (args as any)[2]
                 if (error) {
+                    curError = error
                     tempFailedFiles.push(fileInfo.Key)
                 }
                 onFileFinish?.apply(null, args)
             }
         })
-        if (!tempFailedFiles?.length || times <= 0) return res
-        if (times > 0) {
-            setTimeout(
-                () =>
-                    this.uploadFilesWithRetry({
-                        uploadFiles,
-                        options,
-                        times: times - 1,
-                        interval,
-                        failedFiles: tempFailedFiles
-                    }),
-                interval
-            )
+        // if (!tempFailedFiles?.length || times <= 0) return res
+
+        if (!tempFailedFiles?.length) {
+            return res
+        } else {
+            if (times > 0) {
+                return await new Promise((resolve, reject) => {
+                    setTimeout(
+                        () =>
+                            this.uploadFilesWithRetry({
+                                uploadFiles,
+                                options,
+                                times: times - 1,
+                                interval,
+                                failedFiles: tempFailedFiles
+                            }).then(res => resolve(res))
+                                .catch(err => reject(err)),
+                        interval
+                    )
+                })
+            } else {
+                if (curError) {
+                    throw curError
+                }
+            }
         }
     }