feat: Delete API Details

Author: AbhiPrasad

packages/core/src/api.ts

@@ -1,32 +1,8 @@
-import { DsnComponents, DsnLike, SdkMetadata } from '@sentry/types';
+import { DsnComponents, DsnLike } from '@sentry/types';
 import { dsnToString, makeDsn, urlEncode } from '@sentry/utils';
 
 const SENTRY_API_VERSION = '7';
 
-/**
- * Stores details about a Sentry SDK
- */
-export interface APIDetails {
-  /** The DSN as passed to Sentry.init() */
-  initDsn: DsnLike;
-  /** Metadata about the SDK (name, version, etc) for inclusion in envelope headers */
-  metadata: SdkMetadata;
-  /** The internally used Dsn object. */
-  readonly dsn: DsnComponents;
-  /** The envelope tunnel to use. */
-  readonly tunnel?: string;
-}
-
-/** Initializes API Details */
-export function initAPIDetails(dsn: DsnLike, metadata?: SdkMetadata, tunnel?: string): APIDetails {
-  return {
-    initDsn: dsn,
-    metadata: metadata || {},
-    dsn: makeDsn(dsn),
-    tunnel,
-  } as APIDetails;
-}
-
 /** Returns the prefix to construct Sentry ingestion API endpoints. */
 function getBaseApiEndpoint(dsn: DsnComponents): string {
   const protocol = dsn.protocol ? `${dsn.protocol}:` : '';

packages/core/src/index.ts

@@ -1,4 +1,3 @@
-export type { APIDetails } from './api';
 export type { ClientClass } from './sdk';
 
 export {
@@ -17,7 +16,7 @@ export {
   withScope,
 } from '@sentry/minimal';
 export { addGlobalEventProcessor, getCurrentHub, getHubFromCarrier, Hub, makeMain, Scope, Session } from '@sentry/hub';
-export { getEnvelopeEndpointWithUrlEncodedAuth, initAPIDetails, getReportDialogEndpoint } from './api';
+export { getEnvelopeEndpointWithUrlEncodedAuth, getReportDialogEndpoint } from './api';
 export { BaseClient } from './baseclient';
 export { initAndBind } from './sdk';
 export { createTransport } from './transports/base';

packages/core/test/lib/api.test.ts

@@ -1,21 +1,22 @@
-/* eslint-disable deprecation/deprecation */
 import { makeDsn } from '@sentry/utils';
 
-import { getEnvelopeEndpointWithUrlEncodedAuth, getReportDialogEndpoint, initAPIDetails } from '../../src/api';
+import { getEnvelopeEndpointWithUrlEncodedAuth, getReportDialogEndpoint } from '../../src/api';
 
 const ingestDsn = 'https://abc@xxxx.ingest.sentry.io:1234/subpath/123';
 const dsnPublic = 'https://abc@sentry.io:1234/subpath/123';
 const tunnel = 'https://hello.com/world';
 
-const dsnPublicAPI = initAPIDetails(dsnPublic);
+const dsnPublicComponents = makeDsn(dsnPublic);
 
 describe('API', () => {
   test('getEnvelopeEndpoint', () => {
-    expect(getEnvelopeEndpointWithUrlEncodedAuth(dsnPublicAPI.dsn)).toEqual(
+    expect(getEnvelopeEndpointWithUrlEncodedAuth(dsnPublicComponents)).toEqual(
       'https://sentry.io:1234/subpath/api/123/envelope/?sentry_key=abc&sentry_version=7',
     );
-    const dsnPublicAPIWithTunnel = initAPIDetails(dsnPublic, {}, tunnel);
-    expect(getEnvelopeEndpointWithUrlEncodedAuth(dsnPublicAPIWithTunnel.dsn, tunnel)).toEqual(tunnel);
+  });
+
+  test('getEnvelopeEndpoint with tunnel', () => {
+    expect(getEnvelopeEndpointWithUrlEncodedAuth(dsnPublicComponents, tunnel)).toEqual(tunnel);
   });
 
   describe('getReportDialogEndpoint', () => {
@@ -97,14 +98,4 @@ describe('API', () => {
       },
     );
   });
-
-  test('initAPIDetails dsn', () => {
-    expect(dsnPublicAPI.dsn.host).toEqual(makeDsn(dsnPublic).host);
-    expect(dsnPublicAPI.dsn.path).toEqual(makeDsn(dsnPublic).path);
-    expect(dsnPublicAPI.dsn.pass).toEqual(makeDsn(dsnPublic).pass);
-    expect(dsnPublicAPI.dsn.port).toEqual(makeDsn(dsnPublic).port);
-    expect(dsnPublicAPI.dsn.protocol).toEqual(makeDsn(dsnPublic).protocol);
-    expect(dsnPublicAPI.dsn.projectId).toEqual(makeDsn(dsnPublic).projectId);
-    expect(dsnPublicAPI.dsn.publicKey).toEqual(makeDsn(dsnPublic).publicKey);
-  });
 });

packages/node/test/transports/http.test.ts

@@ -3,11 +3,7 @@ import { EventEnvelope, EventItem } from '@sentry/types';
 import { createEnvelope, serializeEnvelope } from '@sentry/utils';
 import * as http from 'http';
 
-<<<<<<< HEAD
 import { makeNodeTransport } from '../../src/transports';
-=======
-import { makeNodeTransport } from '../../../src/transports';
->>>>>>> 817a118ac (feat: Delete old transports)
 
 jest.mock('@sentry/core', () => {
   const actualCore = jest.requireActual('@sentry/core');