onerway-analytics 1.1.0 → 1.3.0

package/dist/core/analytics.js

@@ -265,10 +265,11 @@ export class Analytics {
             return;
         const apiKey = this.resolveApiKey();
         if (!apiKey) {
-            this.log('No API key configured, dropping events');
-            for (const event of events) {
-                StorageQueue.enqueue(event);
-            }
+            this.log('No API key configured, dropping events (retry disabled)');
+            // HOTFIX (production): localStorage persistence disabled. Original:
+            //      for (const event of events) {
+            //        StorageQueue.enqueue(event);
+            //      }
             return;
         }
         const payload = this.buildPayload(events);
@@ -278,7 +279,15 @@ export class Analytics {
         }
         catch (error) {
             this.log(`Batch send failed (attempt ${retryCount + 1}/${this.config.maxRetryCount + 1}):`, error);
-            if (retryCount < this.config.maxRetryCount) {
+            // ====================================================================
+            // HOTFIX (production): retry / localStorage persistence DISABLED.
+            // Reason: retry loop + parallel recoverFailedEvents + fire-and-forget
+            // setTimeout caused 70+ concurrent in-flight requests on network errors.
+            // DO NOT DELETE the block below — the original implementation must be
+            // restored once the retry mechanism is rewritten (track via ticket).
+            // ====================================================================
+            const retryEnabled = Analytics.RETRY_ENABLED;
+            if (retryEnabled && retryCount < this.config.maxRetryCount) {
                 const delay = 1000 * Math.pow(2, retryCount);
                 this.log(`Retrying in ${delay}ms...`);
                 this.retrying += 1;
@@ -291,10 +300,11 @@ export class Analytics {
                 }, delay);
             }
             else {
-                this.log('Max retries reached, persisting events to localStorage');
-                for (const event of events) {
-                    StorageQueue.enqueue(event);
-                }
+                this.log('Max retries reached / retry disabled, dropping events');
+                // HOTFIX (production): localStorage persistence disabled. Original:
+                //      for (const event of events) {
+                //        StorageQueue.enqueue(event);
+                //      }
             }
         }
     }
@@ -314,6 +324,12 @@ export class Analytics {
             return;
         if (!this.sampled)
             return;
+        if (!Analytics.RETRY_ENABLED) {
+            // HOTFIX (production): storage recovery disabled. Clean any stale items
+            // so they don't accumulate forever.
+            StorageQueue.clear();
+            return;
+        }
         const failedEvents = StorageQueue.recover();
         if (failedEvents.length > 0) {
             this.log(`Recovered ${failedEvents.length} failed events from storage`);
@@ -381,9 +397,11 @@ export class Analytics {
                 }
             }
             catch {
-                for (const event of events) {
-                    StorageQueue.enqueue(event);
-                }
+                // HOTFIX (production): localStorage persistence disabled. Original:
+                //      for (const event of events) {
+                //        StorageQueue.enqueue(event);
+                //      }
+                this.log('Failed to flush buffer on unload, dropping events (retry disabled)');
             }
         };
         this.unloadHandler = handler;
@@ -401,6 +419,12 @@ export class Analytics {
         });
     }
 }
+// HOTFIX (production): retry / recovery / localStorage persistence DISABLED.
+// Reason: retry loop + parallel recoverFailedEvents + fire-and-forget
+// setTimeout caused 70+ concurrent in-flight requests on network errors.
+// Flip this flag to true (and uncomment the StorageQueue.enqueue blocks)
+// to restore the original behavior once the retry mechanism is rewritten.
+Analytics.RETRY_ENABLED = false;
 export function createAnalytics(config) {
     return new Analytics(config);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "onerway-analytics",
-  "version": "1.1.0",
+  "version": "1.3.0",
   "description": "Onrway Analytics SDK",
   "main": "dist/index.js",
   "module": "dist/index.js",