👌 make parameters optional

Author: BenoitZugmeyer

packages/core/src/domain/configuration/endpointBuilder.spec.ts

@@ -173,6 +173,16 @@ describe('endpointBuilder', () => {
       ).toBe('https://quota.proxy.test/api/v2/profiling/quota?session_id=abc')
     })
 
+    it('falls back to empty parameters when calling proxy function', () => {
+      expect(
+        buildEndpointUrl({
+          proxy: ({ path, parameters }) => `https://proxy.test${path}?${parameters}`,
+          site: undefined,
+          path: '/api/v2/rum',
+        })
+      ).toBe('https://proxy.test/api/v2/rum?')
+    })
+
     it('builds intake URL from the site when no proxy is configured', () => {
       expect(
         buildEndpointUrl({

packages/core/src/domain/configuration/endpointBuilder.ts

@@ -58,14 +58,14 @@ export interface BuildEndpointUrlOptions {
   site: Site | undefined
   subdomain?: string
   path: string
-  parameters: string
+  parameters?: string
 }
 
 export function buildEndpointUrl({
   proxy,
   site = INTAKE_SITE_US1,
   path,
-  parameters,
+  parameters = '',
   subdomain,
 }: BuildEndpointUrlOptions): string {
   let pathAndParameters = path

packages/rum-core/src/domain/configuration/remoteConfiguration.ts

@@ -325,7 +325,6 @@ export function buildEndpoint(configuration: RumInitConfiguration) {
     site: configuration.site,
     path: `/${REMOTE_CONFIGURATION_VERSION}/${encodeURIComponent(id)}.json`,
     subdomain: 'sdk-configuration',
-    parameters: '',
   })
 }