fix transport types

Author: AbhiPrasad

packages/browser/src/sdk.ts

@@ -6,13 +6,15 @@ import {
   logger,
   resolvedSyncPromise,
   stackParserFromOptions,
+  supportsFetch,
 } from '@sentry/utils';
 
 import { BrowserClient, BrowserClientOptions, BrowserOptions } from './client';
 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';
 
 export const defaultIntegrations = [
@@ -109,7 +111,8 @@ export function init(options: BrowserOptions = {}): void {
     ...options,
     stackParser: stackParserFromOptions(options),
     integrations: getIntegrationsToSetup(options),
-    transport,
+    // TODO(v7): get rid of transport being passed down below
+    transport: options.transport || (supportsFetch() ? FetchTransport : XHRTransport),
   };
 
   initAndBind(BrowserClient, clientOptions, transport, newTransport);

packages/core/test/lib/base.test.ts

@@ -56,7 +56,7 @@ jest.mock('@sentry/utils', () => {
 function getDefaultTestOptions(options: Partial<TestOptions> = {}): TestOptions {
   return {
     integrations: [],
-    transport: new NoopTransport(),
+    transport: NoopTransport,
     stackParser: () => [],
     ...options,
   };

packages/node/src/sdk.ts

@@ -8,7 +8,7 @@ import { NodeClient } from './client';
 import { IS_DEBUG_BUILD } from './flags';
 import { Console, ContextLines, Http, LinkedErrors, OnUncaughtException, OnUnhandledRejection } from './integrations';
 import { nodeStackParser } from './stack-parser';
-import { setupNodeTransport } from './transports';
+import { HTTPSTransport, HTTPTransport, setupNodeTransport } from './transports';
 import { NodeClientOptions, NodeOptions } from './types';
 
 export const defaultIntegrations = [
@@ -138,7 +138,8 @@ export function init(options: NodeOptions = {}): void {
     ...options,
     stackParser: stackParserFromOptions(options),
     integrations: getIntegrationsToSetup(options),
-    transport,
+    // TODO(v7): Fix me when we switch to new transports entirely.
+    transport: options.transport || (transport instanceof HTTPTransport ? HTTPTransport : HTTPSTransport),
   };
 
   initAndBind(NodeClient, clientOptions, transport, newTransport);
@@ -198,7 +199,7 @@ export function isAutoSessionTrackingEnabled(client?: NodeClient): boolean {
   if (client === undefined) {
     return false;
   }
-  const clientOptions: NodeClientOptions = client && client.getOptions();
+  const clientOptions = client && client.getOptions();
   if (clientOptions && clientOptions.autoSessionTracking !== undefined) {
     return clientOptions.autoSessionTracking;
   }