ref: Switch to new transports (#4943)

This PR switches over from the old transports to the new ones. It deletes the functionality from the client of passing in transports explicitly and instead expects the transport to be passed in through options.

Instead of passing in an instance of a transport to the client, we pass in a constructor function. This allows the client to control exactly what options are passed into the transport.

```ts
this._transport = options.transport({ ...options.transportOptions, url });
```

Author: AbhiPrasad

MIGRATION.md

@@ -115,12 +115,6 @@ For our efforts to reduce bundle size of the SDK we had to remove and refactor p
   [#4919](https://github.com/getsentry/sentry-javascript/pull/4919)). `Backend` was an unnecessary abstraction which is
   not present in other Sentry SDKs. For the sake of reducing complexity, increasing consistency with other Sentry SDKs and
   decreasing bundle-size, `Backend` was removed.
-  <!-- TODO(v7): Add more info and PR link for passing transports in options once this is done -->
-  <!-- TODO(v7): This needs refinement once NewTransport is the default (maybe this should get its own section with an expamp) -->
-- Inject transport into client instead of initializing it in the client in `setupTransport` (see
-  [#4921](https://github.com/getsentry/sentry-javascript/pull/4921/)). If you are creating your own `Client` or
-  calling `initAndBind`, you will have to supply your desired transport. Either provide a custom one or call
-  `setupBrowserTransport` or `setupNodeTransport` for default transports, depending on your requirements.
 - Remove support for Opera browser pre v15
 
 ## Sentry Angular SDK Changes

packages/browser/src/client.ts

@@ -1,5 +1,5 @@
-import { BaseClient, NewTransport, Scope, SDK_VERSION } from '@sentry/core';
-import { ClientOptions, Event, EventHint, Options, Severity, SeverityLevel, Transport } from '@sentry/types';
+import { BaseClient, Scope, SDK_VERSION } from '@sentry/core';
+import { ClientOptions, Event, EventHint, Options, Severity, SeverityLevel } from '@sentry/types';
 import { getGlobalObject, logger } from '@sentry/utils';
 
 import { eventFromException, eventFromMessage } from './eventbuilder';
@@ -47,7 +47,7 @@ export class BrowserClient extends BaseClient<BrowserClientOptions> {
    *
    * @param options Configuration options for this SDK.
    */
-  public constructor(options: BrowserClientOptions, transport: Transport, newTransport?: NewTransport) {
+  public constructor(options: BrowserClientOptions) {
     options._metadata = options._metadata || {};
     options._metadata.sdk = options._metadata.sdk || {
       name: 'sentry.javascript.browser',
@@ -59,7 +59,7 @@ export class BrowserClient extends BaseClient<BrowserClientOptions> {
       ],
       version: SDK_VERSION,
     };
-    super(options, transport, newTransport);
+    super(options);
   }
 
   /**

packages/browser/src/exports.ts

@@ -27,6 +27,7 @@ export {
   captureEvent,
   captureMessage,
   configureScope,
+  createTransport,
   getHubFromCarrier,
   getCurrentHub,
   Hub,

packages/browser/src/sdk.ts

@@ -14,8 +14,7 @@ import { IS_DEBUG_BUILD } from './flags';
 import { ReportDialogOptions, wrap as internalWrap } from './helpers';
 import { Breadcrumbs, Dedupe, GlobalHandlers, LinkedErrors, TryCatch, UserAgent } from './integrations';
 import { defaultStackParsers } from './stack-parsers';
-import { FetchTransport, XHRTransport } from './transports';
-import { setupBrowserTransport } from './transports/setup';
+import { makeNewFetchTransport, makeNewXHRTransport } from './transports';
 
 export const defaultIntegrations = [
   new CoreIntegrations.InboundFilters(),
@@ -102,17 +101,15 @@ export function init(options: BrowserOptions = {}): void {
   if (options.sendClientReports === undefined) {
     options.sendClientReports = true;
   }
-  const { transport, newTransport } = setupBrowserTransport(options);
 
   const clientOptions: BrowserClientOptions = {
     ...options,
     stackParser: stackParserFromOptions(options.stackParser || defaultStackParsers),
     integrations: getIntegrationsToSetup(options),
-    // TODO(v7): get rid of transport being passed down below
-    transport: options.transport || (supportsFetch() ? FetchTransport : XHRTransport),
+    transport: options.transport || (supportsFetch() ? makeNewFetchTransport : makeNewXHRTransport),
   };
 
-  initAndBind(BrowserClient, clientOptions, transport, newTransport);
+  initAndBind(BrowserClient, clientOptions);
 
   if (options.autoSessionTracking) {
     startSessionTracking();

packages/browser/src/transports/index.ts

@@ -4,5 +4,3 @@ export { XHRTransport } from './xhr';
 
 export { makeNewFetchTransport } from './new-fetch';
 export { makeNewXHRTransport } from './new-xhr';
-
-export { setupBrowserTransport } from './setup';

packages/browser/src/transports/new-fetch.ts

@@ -1,10 +1,5 @@
-import {
-  BaseTransportOptions,
-  createTransport,
-  NewTransport,
-  TransportMakeRequestResponse,
-  TransportRequest,
-} from '@sentry/core';
+import { createTransport } from '@sentry/core';
+import { BaseTransportOptions, NewTransport, TransportMakeRequestResponse, TransportRequest } from '@sentry/types';
 
 import { FetchImpl, getNativeFetchImplementation } from './utils';
 

packages/browser/src/transports/new-xhr.ts

@@ -1,10 +1,5 @@
-import {
-  BaseTransportOptions,
-  createTransport,
-  NewTransport,
-  TransportMakeRequestResponse,
-  TransportRequest,
-} from '@sentry/core';
+import { createTransport } from '@sentry/core';
+import { BaseTransportOptions, NewTransport, TransportMakeRequestResponse, TransportRequest } from '@sentry/types';
 import { SyncPromise } from '@sentry/utils';
 
 /**

packages/browser/src/transports/setup.ts

@@ -1,11 +1,12 @@
-import { NoopTransport } from '@sentry/core';
+import { createTransport } from '@sentry/core';
+import { resolvedSyncPromise } from '@sentry/utils';
 
 import { BrowserClientOptions } from '../../../src/client';
 
 export function getDefaultBrowserClientOptions(options: Partial<BrowserClientOptions> = {}): BrowserClientOptions {
   return {
     integrations: [],
-    transport: NoopTransport,
+    transport: () => createTransport({}, _ => resolvedSyncPromise({ statusCode: 200 })),
     stackParser: () => [],
     ...options,
   };

packages/browser/test/unit/helper/browser-client-options.ts

@@ -17,7 +17,7 @@ import {
   wrap,
 } from '../../src';
 import { getDefaultBrowserClientOptions } from './helper/browser-client-options';
-import { SimpleTransport } from './mocks/simpletransport';
+import { makeSimpleTransport } from './mocks/simpletransport';
 
 const dsn = 'https://53039209a22b4ec1bcc296a3c9fdecd6@sentry.io/4291';
 
@@ -31,7 +31,7 @@ describe('SentryBrowser', () => {
     init({
       beforeSend,
       dsn,
-      transport: SimpleTransport,
+      transport: makeSimpleTransport,
     });
   });
 
@@ -77,7 +77,7 @@ describe('SentryBrowser', () => {
     describe('user', () => {
       const EX_USER = { email: 'test@example.com' };
       const options = getDefaultBrowserClientOptions({ dsn });
-      const client = new BrowserClient(options, new SimpleTransport({ dsn }));
+      const client = new BrowserClient(options);
       const reportDialogSpy = jest.spyOn(client, 'showReportDialog');
 
       beforeEach(() => {
@@ -150,7 +150,7 @@ describe('SentryBrowser', () => {
         },
         dsn,
       });
-      getCurrentHub().bindClient(new BrowserClient(options, new SimpleTransport({ dsn })));
+      getCurrentHub().bindClient(new BrowserClient(options));
       captureMessage('test');
     });
 
@@ -164,7 +164,7 @@ describe('SentryBrowser', () => {
         },
         dsn,
       });
-      getCurrentHub().bindClient(new BrowserClient(options, new SimpleTransport({ dsn })));
+      getCurrentHub().bindClient(new BrowserClient(options));
       captureEvent({ message: 'event' });
     });
 
@@ -175,7 +175,7 @@ describe('SentryBrowser', () => {
         dsn,
         integrations: [],
       });
-      getCurrentHub().bindClient(new BrowserClient(options, new SimpleTransport({ dsn })));
+      getCurrentHub().bindClient(new BrowserClient(options));
 
       captureMessage('event222');
       captureMessage('event222');
@@ -192,7 +192,7 @@ describe('SentryBrowser', () => {
         dsn,
         integrations: [new Integrations.InboundFilters({ ignoreErrors: ['capture'] })],
       });
-      getCurrentHub().bindClient(new BrowserClient(options, new SimpleTransport({ dsn })));
+      getCurrentHub().bindClient(new BrowserClient(options));
 
       captureMessage('capture');
 
@@ -244,7 +244,7 @@ describe('SentryBrowser initialization', () => {
     it('should set SDK data when Sentry.init() is called', () => {
       init({ dsn });
 
-      const sdkData = (getCurrentHub().getClient() as any).getTransport()._api.metadata?.sdk;
+      const sdkData = (getCurrentHub().getClient() as any).getOptions()._metadata.sdk;
 
       expect(sdkData?.name).toBe('sentry.javascript.browser');
       expect(sdkData?.packages[0].name).toBe('npm:@sentry/browser');
@@ -254,9 +254,9 @@ describe('SentryBrowser initialization', () => {
 
     it('should set SDK data when instantiating a client directly', () => {
       const options = getDefaultBrowserClientOptions({ dsn });
-      const client = new BrowserClient(options, new SimpleTransport({ dsn }));
+      const client = new BrowserClient(options);
 
-      const sdkData = (client.getTransport() as any)._api.metadata?.sdk;
+      const sdkData = client.getOptions()._metadata?.sdk as any;
 
       expect(sdkData.name).toBe('sentry.javascript.browser');
       expect(sdkData.packages[0].name).toBe('npm:@sentry/browser');
@@ -284,7 +284,7 @@ describe('SentryBrowser initialization', () => {
         },
       });
 
-      const sdkData = (getCurrentHub().getClient() as any).getTransport()._api.metadata?.sdk;
+      const sdkData = (getCurrentHub().getClient() as any).getOptions()._metadata?.sdk;
 
       expect(sdkData.name).toBe('sentry.javascript.angular');
       expect(sdkData.packages[0].name).toBe('npm:@sentry/angular');
@@ -305,7 +305,7 @@ describe('wrap()', () => {
       },
       dsn,
     });
-    getCurrentHub().bindClient(new BrowserClient(options, new SimpleTransport({ dsn })));
+    getCurrentHub().bindClient(new BrowserClient(options));
 
     try {
       wrap(() => {