Modified upload logic and rebuilt dst/index.js

Author: blobles-dev

dist/index.js

@@ -217,12 +217,12 @@ function getTargetFile(targetPath, source) {
 }
 
 async function uploadFile(serverId, targetFile, buffer) {
-  // check if the response was 403 (forbidden), try again until the max retries is reached
   let retries = 0;
   let uploaded = false;
+
   while (!uploaded && retries < 3) {
     try {
-      response = await axios.post(
+      const response = await axios.post(
         `/api/client/servers/${serverId}/files/write`,
         buffer,
         {
@@ -237,18 +237,31 @@ async function uploadFile(serverId, targetFile, buffer) {
           },
         }
       );
-      if (response?.status == 204 || response?.status == 200) {
+
+      // API returns 200 on successful upload
+      if (response.status === 200 || response.status === 204) {
+        core.info(`Successfully uploaded ${targetFile} to ${serverId}`);
         uploaded = true;
       } else {
-        core.error(
-          `Upload failed with status ${response?.status}, retrying...`
+        core.warning(
+          `Upload returned unexpected status ${response.status}, retrying...`
         );
+        retries++;
       }
     } catch (error) {
-      core.error(`Upload failed with error ${error}, retrying...`);
+      retries++;
+      if (retries >= 3) {
+        throw new Error(`Failed to upload ${targetFile} after ${retries} attempts: ${error.message}`);
+      }
+      core.warning(`Upload attempt ${retries} failed, retrying...`);
       core.debug(`Error response: ${JSON.stringify(error?.response?.data)}`);
+      // Wait a bit before retrying
+      await new Promise(resolve => setTimeout(resolve, 1000 * retries));
     }
-    retries++;
+  }
+
+  if (!uploaded) {
+    throw new Error(`Failed to upload ${targetFile} after ${retries} attempts`);
   }
 }