From 1b83afe661b6ab175616617a421dacfa96c40c4e Mon Sep 17 00:00:00 2001
From: Abhijeet Prasad <aprasad@sentry.io>
Date: Fri, 9 Sep 2022 13:39:29 +0200
Subject: [PATCH 1/2] feat(core): Set custom transaction source for event
 processors

---
 packages/core/src/baseclient.ts     | 21 +++++++++++++
 packages/core/test/lib/base.test.ts | 47 +++++++++++++++++++++++++++++
 2 files changed, 68 insertions(+)

diff --git a/packages/core/src/baseclient.ts b/packages/core/src/baseclient.ts
index 6d246106048c..8fef113feb4d 100644
--- a/packages/core/src/baseclient.ts
+++ b/packages/core/src/baseclient.ts
@@ -35,6 +35,7 @@ import {
   SyncPromise,
   truncate,
   uuid4,
+  timestampInSeconds,
 } from '@sentry/utils';
 
 import { getEnvelopeEndpointWithUrlEncodedAuth } from './api';
@@ -658,6 +659,26 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
           this._updateSessionFromEvent(session, processedEvent);
         }
 
+        // None of the Sentry built event processor will update transaction name,
+        // so if the transaction name has been changed by an event processor, we know
+        // it has to come from custom event processor added by a user
+        const transactionInfo = processedEvent.transaction_info;
+        if (isTransaction && transactionInfo && processedEvent.transaction !== event.transaction) {
+          const source = 'custom';
+          processedEvent.transaction_info = {
+            ...transactionInfo,
+            source,
+            changes: [
+              ...transactionInfo.changes,
+              {
+                source,
+                timestamp: timestampInSeconds(),
+                propagations: transactionInfo.propagations,
+              },
+            ],
+          };
+        }
+
         this.sendEvent(processedEvent, hint);
         return processedEvent;
       })
diff --git a/packages/core/test/lib/base.test.ts b/packages/core/test/lib/base.test.ts
index 9c3fcd86f27f..4d0bf265af46 100644
--- a/packages/core/test/lib/base.test.ts
+++ b/packages/core/test/lib/base.test.ts
@@ -1092,6 +1092,53 @@ describe('BaseClient', () => {
       expect(recordLostEventSpy).toHaveBeenCalledWith('event_processor', 'error');
     });
 
+    test('mutating transaction name with event processors sets transaction name change metadata', () => {
+      const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN, enableSend: true });
+      const client = new TestClient(options);
+
+      const transaction: Event = {
+        contexts: {
+          trace: {
+            op: 'pageload',
+            span_id: 'a3df84a60c2e4e76',
+            trace_id: '86f39e84263a4de99c326acab3bfe3bd',
+          },
+        },
+        environment: 'production',
+        event_id: '972f45b826a248bba98e990878a177e1',
+        spans: [],
+        start_timestamp: 1591603196.614865,
+        timestamp: 1591603196.728485,
+        transaction: 'initialName',
+        type: 'transaction',
+        transaction_info: {
+          source: 'url',
+          changes: [],
+          propagations: 3,
+        },
+      };
+
+      const scope = new Scope();
+      scope.addEventProcessor(event => {
+        event.transaction = 'updatedName';
+        return event;
+      });
+
+      client.captureEvent(transaction, {}, scope);
+      expect(TestClient.instance!.event!.transaction).toEqual('updatedName');
+      expect(TestClient.instance!.event!.transaction_info).toEqual({
+        source: 'custom',
+        changes: [
+          {
+            propagations: 3,
+            source: 'custom',
+            timestamp: expect.any(Number),
+          },
+        ],
+        propagations: 3,
+      });
+    });
+
     test('eventProcessor sends an event and logs when it crashes', () => {
       expect.assertions(3);
 

From 6d15645af60831f115e427955094fc7030c4f96b Mon Sep 17 00:00:00 2001
From: Abhijeet Prasad <aprasad@sentry.io>
Date: Tue, 13 Sep 2022 11:16:29 +0200
Subject: [PATCH 2/2] fix lint

---
 packages/core/src/baseclient.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/core/src/baseclient.ts b/packages/core/src/baseclient.ts
index 8fef113feb4d..63140024d071 100644
--- a/packages/core/src/baseclient.ts
+++ b/packages/core/src/baseclient.ts
@@ -33,9 +33,9 @@ import {
   resolvedSyncPromise,
   SentryError,
   SyncPromise,
+  timestampInSeconds,
   truncate,
   uuid4,
-  timestampInSeconds,
 } from '@sentry/utils';
 
 import { getEnvelopeEndpointWithUrlEncodedAuth } from './api';