@statsig/client-core 3.32.4 → 3.32.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@statsig/client-core",
-  "version": "3.32.4",
+  "version": "3.32.5",
   "license": "ISC",
   "homepage": "https://github.com/statsig-io/js-client-monorepo",
   "repository": {

package/src/ErrorBoundary.d.ts

@@ -11,7 +11,7 @@ export declare class ErrorBoundary {
     wrap(instance: unknown, namePrefix?: string): void;
     logError(tag: string, error: unknown): void;
     logDroppedEvents(count: number, reason: string, metadata?: Record<string, unknown>): void;
-    logEventRequestFailure(count: number, reason: string, flushType: string, statusCode: number, retries: number, failurePath?: string): void;
+    logEventRequestFailure(count: number, reason: string, flushType: string, statusCode: number, retries: number, failurePath?: string, failureErrorMessage?: string): void;
     getLastSeenErrorAndReset(): Error | null;
     attachErrorIfNoneExists(error: unknown): void;
     private _capture;

package/src/ErrorBoundary.js

@@ -56,7 +56,7 @@ class ErrorBoundary {
         }
         this._onError(`statsig::log_event_dropped_event_count`, new Error(reason), true, extra);
     }
-    logEventRequestFailure(count, reason, flushType, statusCode, retries, failurePath) {
+    logEventRequestFailure(count, reason, flushType, statusCode, retries, failurePath, failureErrorMessage) {
         const extra = {
             eventCount: String(count),
             flushType: flushType,
@@ -67,6 +67,10 @@ class ErrorBoundary {
         if (failurePath) {
             extra['failurePath'] = failurePath;
         }
+        if (typeof failureErrorMessage === 'string' &&
+            failureErrorMessage.length > 0) {
+            extra['failureErrorMessage'] = failureErrorMessage;
+        }
         this._onError(`statsig::log_event_failed`, new Error(reason), true, extra);
     }
     getLastSeenErrorAndReset() {

package/src/EventSender.d.ts

@@ -7,6 +7,7 @@ type EventSendResult = {
     success: boolean;
     statusCode: number;
     failurePath?: string;
+    failureErrorMessage?: string;
 };
 export declare class EventSender {
     private _network;

package/src/EventSender.js

@@ -55,19 +55,15 @@ class EventSender {
                     });
                     return response;
                 }
-                return {
-                    success: false,
-                    statusCode: response.statusCode,
-                    failurePath: response.failurePath,
-                };
+                return Object.assign({ success: false, statusCode: response.statusCode, failurePath: response.failurePath }, (response.failureErrorMessage
+                    ? { failureErrorMessage: response.failureErrorMessage }
+                    : {}));
             }
             catch (error) {
                 Log_1.Log.warn('Failed to send batch:', error);
-                return {
-                    success: false,
-                    statusCode: -1,
-                    failurePath: (_c = transportFailure.path) !== null && _c !== void 0 ? _c : failurePath,
-                };
+                return Object.assign({ success: false, statusCode: -1, failurePath: (_c = transportFailure.path) !== null && _c !== void 0 ? _c : failurePath }, (transportFailure.errorMessage
+                    ? { failureErrorMessage: transportFailure.errorMessage }
+                    : {}));
             }
         });
     }
@@ -77,13 +73,11 @@ class EventSender {
             const result = yield this._network.post(this._getRequestData(batch), failureInfo);
             const code = (_a = result === null || result === void 0 ? void 0 : result.code) !== null && _a !== void 0 ? _a : -1;
             if (code === -1) {
-                return {
-                    success: false,
-                    statusCode: -1,
-                    failurePath: (_b = failureInfo.path) !== null && _b !== void 0 ? _b : (result === undefined
+                return Object.assign({ success: false, statusCode: -1, failurePath: (_b = failureInfo.path) !== null && _b !== void 0 ? _b : (result === undefined
                         ? 'event_sender_post_returned_undefined'
-                        : 'event_sender_post_returned_null'),
-                };
+                        : 'event_sender_post_returned_null') }, (failureInfo.errorMessage
+                    ? { failureErrorMessage: failureInfo.errorMessage }
+                    : {}));
             }
             return { success: code >= 200 && code < 300, statusCode: code };
         });
@@ -91,13 +85,11 @@ class EventSender {
     _sendEventsViaBeacon(batch, failureInfo) {
         var _a;
         const success = this._network.beacon(this._getRequestData(batch), failureInfo);
-        return {
-            success,
-            statusCode: success ? 200 : -1,
-            failurePath: success
+        return Object.assign({ success, statusCode: success ? 200 : -1, failurePath: success
                 ? undefined
-                : (_a = failureInfo.path) !== null && _a !== void 0 ? _a : 'beacon_send_false',
-        };
+                : (_a = failureInfo.path) !== null && _a !== void 0 ? _a : 'beacon_send_false' }, (!success && failureInfo.errorMessage
+            ? { failureErrorMessage: failureInfo.errorMessage }
+            : {}));
     }
     _getRequestData(batch) {
         return {

package/src/FlushCoordinator.d.ts

@@ -44,6 +44,7 @@ export declare class FlushCoordinator {
     private _prepareQueueForFlush;
     containsAtLeastOneFullBatch(): boolean;
     convertPendingEventsToBatches(): number;
+    private _isRetryableBatch;
     private _handleFailure;
     loadAndRetryShutdownFailedEvents(): Promise<void>;
     private _getStorageKey;

package/src/FlushCoordinator.js

@@ -21,6 +21,13 @@ const SafeJs_1 = require("./SafeJs");
 const SessionID_1 = require("./SessionID");
 const StatsigOptionsCommon_1 = require("./StatsigOptionsCommon");
 const StorageProvider_1 = require("./StorageProvider");
+const RETRYABLE_NO_RESPONSE_FAILURE_PATHS = new Set([
+    'network_request_timed_out_no_response',
+    'network_request_exception_no_response',
+    'event_sender_post_returned_null',
+    'event_sender_post_returned_undefined',
+    'event_sender_post_exception',
+]);
 class FlushCoordinator {
     constructor(batchQueue, pendingEvents, onPrepareFlush, 
     // For Event Sender
@@ -254,7 +261,7 @@ class FlushCoordinator {
                 return true;
             }
             this._flushInterval.adjustForFailure();
-            this._handleFailure(batch, flushType, result.statusCode, result.failurePath);
+            this._handleFailure(batch, flushType, result.statusCode, result.failurePath, result.failureErrorMessage);
             return false;
         });
     }
@@ -283,23 +290,34 @@ class FlushCoordinator {
         const allEvents = this._pendingEvents.takeAll();
         return this._batchQueue.createBatches(allEvents);
     }
-    _handleFailure(batch, flushType, statusCode, failurePath) {
+    _isRetryableBatch(statusCode, failurePath) {
+        if (NetworkCore_1.RETRYABLE_CODES.has(statusCode)) {
+            return true;
+        }
+        if (statusCode === -1 &&
+            failurePath &&
+            RETRYABLE_NO_RESPONSE_FAILURE_PATHS.has(failurePath)) {
+            return true;
+        }
+        return false;
+    }
+    _handleFailure(batch, flushType, statusCode, failurePath, failureErrorMessage) {
         if (flushType === FlushTypes_1.FlushType.Shutdown) {
             Log_1.Log.warn(`${flushType} flush failed during shutdown. ` +
                 `${batch.events.length} event(s) will be saved to storage for retry in next session.`);
             this._saveShutdownFailedEventsToStorage(batch.events);
             return;
         }
-        if (!NetworkCore_1.RETRYABLE_CODES.has(statusCode)) {
+        if (!this._isRetryableBatch(statusCode, failurePath)) {
             Log_1.Log.warn(`${flushType} flush failed after ${batch.attempts} attempt(s). ` +
                 `${batch.events.length} event(s) will be dropped. Non-retryable error: ${statusCode}`);
-            this._errorBoundary.logEventRequestFailure(batch.events.length, `non-retryable error`, flushType, statusCode, batch.attempts, failurePath);
+            this._errorBoundary.logEventRequestFailure(batch.events.length, `non-retryable error`, flushType, statusCode, batch.attempts, failurePath, failureErrorMessage);
             return;
         }
         if (batch.attempts >= EventRetryConstants_1.EventRetryConstants.MAX_RETRY_ATTEMPTS) {
             Log_1.Log.warn(`${flushType} flush failed after ${batch.attempts} attempt(s). ` +
                 `${batch.events.length} event(s) will be dropped.`);
-            this._errorBoundary.logEventRequestFailure(batch.events.length, `max retry attempts exceeded`, flushType, statusCode, batch.attempts, failurePath);
+            this._errorBoundary.logEventRequestFailure(batch.events.length, `max retry attempts exceeded`, flushType, statusCode, batch.attempts, failurePath, failureErrorMessage);
             return;
         }
         batch.incrementAttempts();

package/src/NetworkCore.d.ts

@@ -26,6 +26,7 @@ export type RequestArgsWithData = Flatten<RequestArgs & {
 } & DataFlags>;
 export type RequestFailureInfo = {
     path?: string;
+    errorMessage?: string;
 };
 type BeaconRequestArgs = Pick<RequestArgsWithData, 'data' | 'sdkKey' | 'urlConfig' | 'params' | 'isCompressable' | 'attempt'>;
 type NetworkResponse = {

package/src/NetworkCore.js

@@ -210,6 +210,9 @@ class NetworkCore {
                             failureInfo.path = timedOut
                                 ? 'network_request_timed_out_no_response'
                                 : 'network_request_exception_no_response';
+                            if (errorMessage) {
+                                failureInfo.errorMessage = errorMessage;
+                            }
                         }
                     }
                     return null;

package/src/StatsigMetadata.d.ts

@@ -1,4 +1,4 @@
-export declare const SDK_VERSION = "3.32.4";
+export declare const SDK_VERSION = "3.32.5";
 export type StatsigMetadata = {
     readonly [key: string]: string | undefined | null;
     readonly appVersion?: string;

package/src/StatsigMetadata.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.StatsigMetadataProvider = exports.SDK_VERSION = void 0;
-exports.SDK_VERSION = '3.32.4';
+exports.SDK_VERSION = '3.32.5';
 let metadata = {
     sdkVersion: exports.SDK_VERSION,
     sdkType: 'js-mono', // js-mono is overwritten by Precomp and OnDevice clients