qase-javascript-commons 2.4.11 → 2.4.12

package/changelog.md

@@ -1,3 +1,9 @@
+# qase-javascript-commons@2.4.12
+
+## What's new
+
+Improved the retry mechanism for uploading attachments.
+
 # qase-javascript-commons@2.4.11
 
 ## What's new

package/dist/client/clientV1.js

@@ -145,6 +145,12 @@ class ClientV1 {
             return [];
         }
         const uploadedHashes = [];
+        // Add initial random delay to spread out requests from different workers/shard
+        // This helps prevent all workers from hitting the API at the same time
+        if (attachments.length > 0) {
+            const initialJitter = Math.random() * 500; // 0-500ms random delay
+            await this.delay(initialJitter);
+        }
         for (let i = 0; i < attachments.length; i++) {
             const attachment = attachments[i];
             if (!attachment) {
@@ -165,7 +171,10 @@ class ClientV1 {
             // Add delay between requests to avoid rate limiting
             // Skip delay after the last attachment
             if (i < attachments.length - 1) {
-                await this.delay(100); // 100ms delay between requests
+                // Increased delay with random jitter to prevent synchronization
+                const baseDelay = 1000; // 1000ms (1 second) base delay
+                const jitter = Math.random() * 300; // 0-300ms random jitter
+                await this.delay(baseDelay + jitter);
             }
         }
         return uploadedHashes;
@@ -193,7 +202,12 @@ class ClientV1 {
                     if (error.response?.status === 429) {
                         if (attempt < maxRetries) {
                             const retryAfter = this.getRetryAfter(error);
-                            const waitTime = retryAfter ?? delay;
+                            const baseWaitTime = retryAfter ?? delay;
+                            // Add jitter (random delay) to prevent all workers from retrying simultaneously
+                            // Jitter is 10-30% of the wait time to spread out retry attempts
+                            const jitterPercent = 0.1 + Math.random() * 0.2; // 10-30%
+                            const jitter = baseWaitTime * jitterPercent;
+                            const waitTime = Math.floor(baseWaitTime + jitter);
                             this.logger.logDebug(`Rate limit exceeded (429) for attachment "${attachmentName}". ` +
                                 `Retrying in ${waitTime}ms (attempt ${attempt + 1}/${maxRetries})`);
                             await this.delay(waitTime);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qase-javascript-commons",
-  "version": "2.4.11",
+  "version": "2.4.12",
   "description": "Qase JS Reporters",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",