ref: DSN to Dsn casing change (#1503)

* ref: Rename any DSN occurence to Dsn
* ref: User API to filter sentry requests in breadcrumbs

Author: kamilogorek

packages/browser/src/backend.ts

@@ -44,8 +44,8 @@ export class BrowserBackend implements Backend {
    * @inheritDoc
    */
   public install(): boolean {
-    // We are only called by the client if the SDK is enabled and a valid DSN
-    // has been configured. If no DSN is present, this indicates a programming
+    // We are only called by the client if the SDK is enabled and a valid Dsn
+    // has been configured. If no Dsn is present, this indicates a programming
     // error.
     const dsn = this.options.dsn;
     if (!dsn) {
@@ -140,8 +140,8 @@ export class BrowserBackend implements Backend {
    */
   public async sendEvent(event: SentryEvent): Promise<SentryResponse> {
     if (!this.options.dsn) {
-      logger.warn(`Event has been skipped because no DSN is configured.`);
-      // We do nothing in case there is no DSN
+      logger.warn(`Event has been skipped because no Dsn is configured.`);
+      // We do nothing in case there is no Dsn
       return { status: Status.Skipped };
     }
 

packages/browser/src/client.ts

@@ -1,5 +1,5 @@
 import { API, BaseClient, SentryError } from '@sentry/core';
-import { DSNLike } from '@sentry/types';
+import { DsnLike } from '@sentry/types';
 import { getGlobalObject } from '@sentry/utils/misc';
 import { BrowserBackend, BrowserOptions } from './backend';
 
@@ -23,7 +23,7 @@ export class BrowserClient extends BaseClient<BrowserBackend, BrowserOptions> {
   public showReportDialog(options: {
     [key: string]: any;
     eventId?: string;
-    dsn?: DSNLike;
+    dsn?: DsnLike;
     user?: {
       email?: string;
       name?: string;
@@ -47,14 +47,14 @@ export class BrowserClient extends BaseClient<BrowserBackend, BrowserOptions> {
       return;
     }
 
-    const dsn = options.dsn || this.getDSN();
+    const dsn = options.dsn || this.getDsn();
 
     if (!options.eventId) {
       throw new SentryError('Missing `eventId` option in showReportDialog call');
     }
 
     if (!dsn) {
-      throw new SentryError('Missing `DSN` option in showReportDialog call');
+      throw new SentryError('Missing `Dsn` option in showReportDialog call');
     }
 
     const script = document.createElement('script');

packages/browser/src/integrations/breadcrumbs.ts

@@ -1,4 +1,4 @@
-import { DSN } from '@sentry/core';
+import { API } from '@sentry/core';
 import { getCurrentHub } from '@sentry/hub';
 import { Integration, Severity } from '@sentry/types';
 import { isFunction, isString } from '@sentry/utils/is';
@@ -354,8 +354,7 @@ export class Breadcrumbs implements Integration {
    * Can be disabled or individually configured via the `autoBreadcrumbs` config option
    */
   public install(options: BrowserOptions = {}): void {
-    // TODO: Use API provider instead of raw `new DSN`
-    const filterUrl = options.dsn && new DSN(options.dsn).user;
+    const filterUrl = options.dsn && new API(options.dsn).getStoreEndpoint();
 
     if (this.config.console) {
       this.instrumentConsole();

packages/browser/src/sdk.ts

@@ -32,7 +32,7 @@ export const defaultIntegrations = [
  * import { init } from '@sentry/browser';
  *
  * init({
- *   dsn: '__DSN__',
+ *   dsn: '__Dsn__',
  *   // ...
  * });
  *

packages/browser/test/backend.test.ts

@@ -24,13 +24,13 @@ let backend: BrowserBackend;
 
 describe('BrowserBackend', () => {
   describe('sendEvent()', () => {
-    it('should throw when no DSN is provided', async () => {
+    it('should throw when no Dsn is provided', async () => {
       backend = new BrowserBackend({ dsn });
 
       try {
         await backend.sendEvent(testEvent);
       } catch (e) {
-        expect(e.message).equal('Cannot sendEvent without a valid DSN');
+        expect(e.message).equal('Cannot sendEvent without a valid Dsn');
       }
     });
 

packages/browser/test/transports/base.test.ts

@@ -1,13 +1,13 @@
 import { expect } from 'chai';
 import { BaseTransport } from '../../src/transports/base';
 
-const testDSN = 'https://[email protected]/42';
+const testDsn = 'https://[email protected]/42';
 
 class SimpleTransport extends BaseTransport {}
 
 describe('BaseTransport', () => {
   it('doesnt provide send() implementation', async () => {
-    const transport = new SimpleTransport({ dsn: testDSN });
+    const transport = new SimpleTransport({ dsn: testDsn });
 
     try {
       await transport.send({});
@@ -17,7 +17,7 @@ describe('BaseTransport', () => {
   });
 
   it('has correct endpoint url', () => {
-    const transport = new SimpleTransport({ dsn: testDSN });
+    const transport = new SimpleTransport({ dsn: testDsn });
     expect(transport.url).equal('https://sentry.io/api/42/store/?sentry_key=123&sentry_version=7');
   });
 });

packages/browser/test/transports/beacon.test.ts

@@ -2,7 +2,7 @@ import { expect } from 'chai';
 import { SinonStub, stub } from 'sinon';
 import { Status, Transports } from '../../src';
 
-const testDSN = 'https://[email protected]/42';
+const testDsn = 'https://[email protected]/42';
 const transportUrl = 'https://sentry.io/api/42/store/?sentry_key=123&sentry_version=7';
 const payload = {
   event_id: '1337',
@@ -18,7 +18,7 @@ let transport: Transports.BaseTransport;
 describe('BeaconTransport', () => {
   beforeEach(() => {
     sendBeacon = stub(window.navigator, 'sendBeacon');
-    transport = new Transports.BeaconTransport({ dsn: testDSN });
+    transport = new Transports.BeaconTransport({ dsn: testDsn });
   });
 
   afterEach(() => {

packages/browser/test/transports/fetch.test.ts

@@ -2,7 +2,7 @@ import { expect } from 'chai';
 import { SinonStub, stub } from 'sinon';
 import { Status, Transports } from '../../src';
 
-const testDSN = 'https://[email protected]/42';
+const testDsn = 'https://[email protected]/42';
 const transportUrl = 'https://sentry.io/api/42/store/?sentry_key=123&sentry_version=7';
 const payload = {
   event_id: '1337',
@@ -18,7 +18,7 @@ let transport: Transports.BaseTransport;
 describe('FetchTransport', () => {
   beforeEach(() => {
     fetch = stub(window, 'fetch');
-    transport = new Transports.FetchTransport({ dsn: testDSN });
+    transport = new Transports.FetchTransport({ dsn: testDsn });
   });
 
   afterEach(() => {

packages/browser/test/transports/xhr.test.ts

@@ -2,7 +2,7 @@ import { expect } from 'chai';
 import { fakeServer, SinonFakeServer } from 'sinon';
 import { Status, Transports } from '../../src';
 
-const testDSN = 'https://[email protected]/42';
+const testDsn = 'https://[email protected]/42';
 const transportUrl = 'https://sentry.io/api/42/store/?sentry_key=123&sentry_version=7';
 const payload = {
   event_id: '1337',
@@ -19,7 +19,7 @@ describe('XHRTransport', () => {
   beforeEach(() => {
     server = fakeServer.create();
     server.respondImmediately = true;
-    transport = new Transports.XHRTransport({ dsn: testDSN });
+    transport = new Transports.XHRTransport({ dsn: testDsn });
   });
 
   afterEach(() => {

packages/core/src/api.ts

@@ -1,20 +1,20 @@
-import { DSNLike } from '@sentry/types';
+import { DsnLike } from '@sentry/types';
 import { urlEncode } from '@sentry/utils/object';
-import { DSN } from './dsn';
+import { Dsn } from './dsn';
 
 const SENTRY_API_VERSION = '7';
 
 /** Helper class to provide urls to different Sentry endpoints. */
 export class API {
-  /** The internally used DSN object. */
-  private readonly dsnObject: DSN;
+  /** The internally used Dsn object. */
+  private readonly dsnObject: Dsn;
   /** Create a new instance of API */
-  public constructor(public dsn: DSNLike) {
-    this.dsnObject = new DSN(dsn);
+  public constructor(public dsn: DsnLike) {
+    this.dsnObject = new Dsn(dsn);
   }
 
-  /** Returns the DSN object. */
-  public getDSN(): DSN {
+  /** Returns the Dsn object. */
+  public getDsn(): Dsn {
     return this.dsnObject;
   }