ref: Remove tracingOrigins options (#10614)

Author: Luca Forstner

MIGRATION.md

@@ -1,5 +1,11 @@
 # Upgrading from 7.x to 8.x
 
+## Removal of the `tracingOrigins` option
+
+After its deprecation in v7 the `tracingOrigins` option is now removed in favor of the `tracePropagationTargets` option.
+The `tracePropagationTargets` option should be set in the `Sentry.init()` options, or in your custom `Client`s option if
+you create them. The `tracePropagationTargets` option can no longer be set in the `browserTracingIntegration()` options.
+
 ## Dropping Support for React 15
 
 Sentry will no longer officially support React 15 in version 8. This means that React 15.x will be removed

dev-packages/browser-integration-tests/loader-suites/loader/onLoad/customBrowserTracing/init.js

@@ -4,9 +4,8 @@ Sentry.onLoad(function () {
   Sentry.init({
     integrations: [
       // Without this syntax, this will be re-written by the test framework
-      new window['Sentry'].BrowserTracing({
-        tracePropagationTargets: ['http://localhost:1234'],
-      }),
+      window['Sentry'].browserTracingIntegration(),
     ],
+    tracePropagationTargets: ['http://localhost:1234'],
   });
 });

dev-packages/browser-integration-tests/loader-suites/loader/onLoad/customBrowserTracing/test.ts

@@ -25,11 +25,4 @@ sentryTest('should handle custom added BrowserTracing integration', async ({ get
   expect(eventData.contexts?.trace?.op).toBe('pageload');
   expect(eventData.spans?.length).toBeGreaterThan(0);
   expect(eventData.transaction_info?.source).toEqual('url');
-
-  const tracePropagationTargets = await page.evaluate(() => {
-    const browserTracing = (window as any).Sentry.getClient().getIntegrationByName('BrowserTracing');
-    return browserTracing.options.tracePropagationTargets;
-  });
-
-  expect(tracePropagationTargets).toEqual(['http://localhost:1234']);
 });

dev-packages/browser-integration-tests/suites/replay/dsc/init.js

@@ -1,5 +1,4 @@
 import * as Sentry from '@sentry/browser';
-import { Integrations } from '@sentry/tracing';
 
 window.Sentry = Sentry;
 window.Replay = new Sentry.Replay({
@@ -11,7 +10,8 @@ window.Replay = new Sentry.Replay({
 
 Sentry.init({
   dsn: 'https://[email protected]/1337',
-  integrations: [new Integrations.BrowserTracing({ tracingOrigins: [/.*/] }), window.Replay],
+  integrations: [Sentry.browserTracingIntegration(), window.Replay],
+  tracePropagationTargets: [/.*/],
   environment: 'production',
   tracesSampleRate: 1,
   // Needs manual start!

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/tracePropagationTargets/customTargets/init.js

@@ -4,6 +4,7 @@ window.Sentry = Sentry;
 
 Sentry.init({
   dsn: 'https://[email protected]/1337',
-  integrations: [Sentry.browserTracingIntegration({ tracePropagationTargets: ['http://example.com'] })],
+  integrations: [Sentry.browserTracingIntegration()],
+  tracePropagationTargets: ['http://example.com'],
   tracesSampleRate: 1,
 });

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/tracePropagationTargets/customTargetsAndOrigins/init.js

@@ -1,10 +1,10 @@
 import * as Sentry from '@sentry/browser';
-import { Integrations } from '@sentry/tracing';
 
 window.Sentry = Sentry;
 
 Sentry.init({
   dsn: 'https://[email protected]/1337',
-  integrations: [new Integrations.BrowserTracing({ tracePropagationTargets: ['http://example.com'] })],
+  integrations: [Sentry.browserTracingIntegration()],
+  tracePropagationTargets: ['http://example.com'],
   tracesSampleRate: 1,
 });

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/tracePropagationTargets/customTargetsAndOrigins/subject.js

@@ -1,10 +1,9 @@
 import * as Sentry from '@sentry/browser';
-import { Integrations } from '@sentry/tracing';
 
 window.Sentry = Sentry;
 
 Sentry.init({
   dsn: 'https://[email protected]/1337',
-  integrations: [new Integrations.BrowserTracing()],
+  integrations: [Sentry.browserTracingIntegration()],
   tracesSampleRate: 1,
 });

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/tracePropagationTargets/customTargetsAndOrigins/test.ts

@@ -1,11 +1,11 @@
 import * as Sentry from '@sentry/browser';
-import { Integrations } from '@sentry/tracing';
 
 window.Sentry = Sentry;
 
 Sentry.init({
   dsn: 'https://[email protected]/1337',
-  integrations: [new Integrations.BrowserTracing({ tracingOrigins: [/.*/] })],
+  integrations: [Sentry.browserTracingIntegration()],
+  tracePropagationTargets: [/.*/],
   environment: 'production',
   tracesSampleRate: 1,
   debug: true,

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/tracePropagationTargets/customTracingOrigins/init.js

@@ -5,6 +5,7 @@ window.Sentry = Sentry;
 Sentry.init({
   dsn: 'https://[email protected]/1337',
   // disable pageload transaction
-  integrations: [Sentry.BrowserTracing({ tracingOrigins: ['http://example.com'], startTransactionOnPageLoad: false })],
+  integrations: [Sentry.browserTracingIntegration({ instrumentPageLoad: false })],
+  tracePropagationTargets: ['http://example.com'],
   tracesSampleRate: 1,
 });

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/tracePropagationTargets/customTracingOrigins/subject.js

@@ -1,10 +1,10 @@
 import * as Sentry from '@sentry/browser';
-import { Integrations } from '@sentry/tracing';
 
 window.Sentry = Sentry;
 
 Sentry.init({
   dsn: 'https://[email protected]/1337',
-  integrations: [new Integrations.BrowserTracing({ tracingOrigins: ['http://example.com'] })],
+  integrations: [Sentry.browserTracingIntegration()],
+  tracePropagationTargets: ['http://example.com'],
   tracesSampleRate: 1,
 });

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/tracePropagationTargets/customTracingOrigins/test.ts

@@ -1,10 +1,10 @@
 import * as Sentry from '@sentry/browser';
-import { Integrations } from '@sentry/tracing';
 
 window.Sentry = Sentry;
 
 Sentry.init({
   dsn: 'https://[email protected]/1337',
-  integrations: [new Integrations.BrowserTracing({ tracingOrigins: ['http://example.com'] })],
+  integrations: [Sentry.browserTracingIntegration()],
+  tracePropagationTargets: ['http://example.com'],
   tracesSampleRate: 1,
 });

dev-packages/browser-integration-tests/suites/tracing/browsertracing/tracePropagationTargets/customTargets/init.js

@@ -5,6 +5,7 @@ window.Sentry = Sentry;
 Sentry.init({
   dsn: 'https://[email protected]/1337',
   // disable pageload transaction
-  integrations: [Sentry.BrowserTracing({ tracingOrigins: ['http://example.com'], startTransactionOnPageLoad: false })],
+  integrations: [Sentry.browserTracingIntegration({ instrumentPageLoad: false })],
+  tracePropagationTargets: ['http://example.com'],
   tracesSampleRate: 1,
 });

dev-packages/browser-integration-tests/suites/tracing/browsertracing/tracePropagationTargets/customTargetsAndOrigins/init.js

@@ -24,8 +24,8 @@ module.exports = function (environment) {
       tracesSampleRate: 1,
       // Include fake dsn so that instrumentation is enabled when running from cli
       dsn: process.env.SENTRY_DSN || 'https://[email protected]/0',
+      tracePropagationTargets: ['localhost', 'doesntexist.example'],
       browserTracingOptions: {
-        tracingOrigins: ['localhost', 'doesntexist.example'],
         _experiments: {
           // This lead to some flaky tests, as that is sometimes logged
           enableLongTask: false,

dev-packages/browser-integration-tests/suites/tracing/browsertracing/tracePropagationTargets/customTargetsAndOrigins/subject.js

@@ -58,7 +58,7 @@ describe('browserTracingIntegration', () => {
       };
     });
 
-  const fakeClient = { getOptions: () => undefined, on: () => {} };
+  const fakeClient = { getOptions: () => ({}), on: () => {} };
 
   const mockedRoutingSpan = {
     end: () => {},