fix: Do not throw from newDefaultEventDispatcher on invalid key

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eppo/js-client-sdk-common",
-  "version": "4.5.0",
+  "version": "4.5.1",
   "description": "Eppo SDK for client-side JavaScript applications (base for both web and react native)",
   "main": "dist/index.js",
   "files": [

src/events/default-event-dispatcher.spec.ts

@@ -6,12 +6,13 @@
 } from './default-event-dispatcher';
 import { Event } from './event-dispatcher';
 import NetworkStatusListener from './network-status-listener';
+import NoOpEventDispatcher from './no-op-event-dispatcher';
 
 global.fetch = jest.fn();
 
 const mockNetworkStatusListener = {
   isOffline: () => false,
   onNetworkStatusChange: (_: (_: boolean) => void) => null as unknown as void,
 };
 
 const createDispatcher = (
@@ -141,7 +142,7 @@
   describe('offline handling', () => {
     it('skips delivery when offline', async () => {
       let isOffline = false;
       let cb = (_: boolean) => null as unknown as void;
       const networkStatusListener = {
         isOffline: () => isOffline,
         onNetworkStatusChange: (callback: (isOffline: boolean) => void) => {
@@ -165,7 +166,7 @@
 
     it('resumes delivery when back online', async () => {
       let isOffline = true;
       let cb = (_: boolean) => null as unknown as void;
       const networkStatusListener = {
         isOffline: () => isOffline,
         onNetworkStatusChange: (callback: (isOffline: boolean) => void) => {
@@ -201,14 +202,13 @@
   });
 
   describe('newDefaultEventDispatcher', () => {
-    it('should throw if SDK key is invalid', () => {
-      expect(() => {
-        newDefaultEventDispatcher(
-          new ArrayBackedNamedEventQueue('test-queue'),
-          mockNetworkStatusListener,
-          'invalid-sdk-key',
-        );
-      }).toThrow('Unable to parse Event ingestion URL from SDK key');
+    it('should fallback to no-op dispatcher if SDK key is invalid', () => {
+      const eventDispatcher = newDefaultEventDispatcher(
+        new ArrayBackedNamedEventQueue('test-queue'),
+        mockNetworkStatusListener,
+        'invalid-sdk-key',
+      );
+      expect(eventDispatcher).toBeInstanceOf(NoOpEventDispatcher);
     });
 
     it('should create a new DefaultEventDispatcher with the provided configuration', () => {

src/events/default-event-dispatcher.ts

@@ -6,6 +6,7 @@ import EventDelivery from './event-delivery';
 import EventDispatcher, { Event } from './event-dispatcher';
 import NamedEventQueue from './named-event-queue';
 import NetworkStatusListener from './network-status-listener';
+import NoOpEventDispatcher from './no-op-event-dispatcher';
 import SdkKeyDecoder from './sdk-key-decoder';
 
 export type EventDispatcherConfig = {
@@ -135,7 +136,10 @@ export function newDefaultEventDispatcher(
   const sdkKeyDecoder = new SdkKeyDecoder();
   const ingestionUrl = sdkKeyDecoder.decodeEventIngestionHostName(sdkKey);
   if (!ingestionUrl) {
-    throw new Error('Unable to parse Event ingestion URL from SDK key');
+    logger.debug(
+      'Unable to parse Event ingestion URL from SDK key, falling back to no-op event dispatcher',
+    );
+    return new NoOpEventDispatcher();
   }
   return new DefaultEventDispatcher(
     new BatchEventProcessor(eventQueue, batchSize),