✨ [RUM-11701] Add unity as a source override (#3838)

* ✨ [RUM-11701] Add unity as a source override

Source overrides only currently support Flutter, and should support Unity as a source override as well.

* Update packages/core/src/domain/configuration/transportConfiguration.ts

Co-authored-by: Benoît <[email protected]>

* Update packages/core/src/domain/configuration/transportConfiguration.ts

Co-authored-by: Adrian de la Rosa <[email protected]>

---------

Co-authored-by: Benoît <[email protected]>
Co-authored-by: Adrian de la Rosa <[email protected]>

Author: 3 people

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

@@ -223,7 +223,7 @@ export interface InitConfiguration {
    *
    * @internal
    */
-  source?: 'browser' | 'flutter' | undefined
+  source?: 'browser' | 'flutter' | 'unity' | undefined
 
   /**
    * [Internal option] Additional configuration for the SDK.
@@ -286,7 +286,7 @@ export interface Configuration extends TransportConfiguration {
 
   // internal
   sdkVersion: string | undefined
-  source: 'browser' | 'flutter'
+  source: 'browser' | 'flutter' | 'unity'
   variant: string | undefined
 }
 

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

@@ -158,10 +158,16 @@ describe('endpointBuilder', () => {
       expect(endpoint).toContain('ddsource=browser')
     })
 
-    it('should source when provided', () => {
+    it('should use flutter source when provided', () => {
       const config = { ...initConfiguration, source: 'flutter' as const }
       const endpoint = createEndpointBuilder(config, 'rum').build('fetch', DEFAULT_PAYLOAD)
       expect(endpoint).toContain('ddsource=flutter')
     })
+
+    it('should use unity source when provided', () => {
+      const config = { ...initConfiguration, source: 'unity' as const }
+      const endpoint = createEndpointBuilder(config, 'rum').build('fetch', DEFAULT_PAYLOAD)
+      expect(endpoint).toContain('ddsource=unity')
+    })
   })
 })

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

@@ -13,7 +13,7 @@ export interface TransportConfiguration {
   datacenter?: string | undefined
   replica?: ReplicaConfiguration
   site: Site
-  source: 'browser' | 'flutter'
+  source: 'browser' | 'flutter' | 'unity'
 }
 
 export interface ReplicaConfiguration {
@@ -23,7 +23,7 @@ export interface ReplicaConfiguration {
 
 export function computeTransportConfiguration(initConfiguration: InitConfiguration): TransportConfiguration {
   const site = initConfiguration.site || INTAKE_SITE_US1
-  const source = initConfiguration.source === 'flutter' ? 'flutter' : 'browser'
+  const source = validateSource(initConfiguration.source)
 
   const endpointBuilders = computeEndpointBuilders({ ...initConfiguration, site, source })
   const replicaConfiguration = computeReplicaConfiguration({ ...initConfiguration, site, source })
@@ -36,6 +36,13 @@ export function computeTransportConfiguration(initConfiguration: InitConfigurati
   }
 }
 
+function validateSource(source: string | undefined) {
+  if (source === 'flutter' || source === 'unity') {
+    return source
+  }
+  return 'browser'
+}
+
 function computeEndpointBuilders(initConfiguration: InitConfiguration) {
   return {
     logsEndpointBuilder: createEndpointBuilder(initConfiguration, 'logs'),