fix: wrong events path for clientSideID

launchdarkly · Aug 26, 2023 · 37bdbf4 · 37bdbf4
1 parent cada2f6
commit 37bdbf4
Showing 1 changed file with 5 additions and 8 deletions.
diff --git a/packages/shared/sdk-server/src/events/EventSender.ts b/packages/shared/sdk-server/src/events/EventSender.ts
@@ -40,15 +40,12 @@ export default class EventSender implements subsystem.LDEventSender {
       ...defaultHeaders(sdkKey, config, info),
     };
 
-    this.eventsUri = `${events}/bulk`;
+    const isClientSideID = !sdkKey.startsWith('sdk-');
+    this.eventsUri = isClientSideID ? `${events}/events/bulk/${sdkKey}` : `${events}/bulk`;
+    this.diagnosticEventsUri = isClientSideID
+      ? `${events}/events/diagnostic/${sdkKey}`
+      : `${events}/diagnostic`;
 
-    // edge sdks use clientSideID so we use the environment endpoint
-    const useEnvironmentEndpoint = !sdkKey.startsWith('sdk-');
-    if (useEnvironmentEndpoint) {
-      this.eventsUri = `${this.eventsUri}/${sdkKey}`;
-    }
-
-    this.diagnosticEventsUri = `${events}/diagnostic`;
     this.requests = requests;
     this.crypto = crypto;
   }