ref: Don't wrap xhr/fetch for sentry breadcrumbs

HazAT · HazAT · commit 8387ed3232aa · 2020-05-26T15:00:27.000+02:00
diff --git a/packages/browser/src/client.ts b/packages/browser/src/client.ts
@@ -3,6 +3,7 @@ import { DsnLike, Event, EventHint } from '@sentry/types';
 import { getGlobalObject, logger } from '@sentry/utils';
 
 import { BrowserBackend, BrowserOptions } from './backend';
+import { Breadcrumbs } from './integrations/index';
 import { SDK_NAME, SDK_VERSION } from './version';
 
 /**
@@ -69,6 +70,35 @@ export class BrowserClient extends BaseClient<BrowserBackend, BrowserOptions> {
     return super._prepareEvent(event, scope, hint);
   }
 
+  /**
+   * @inheritDoc
+   */
+  public captureEvent(event: Event, hint?: EventHint, scope?: Scope): string | undefined {
+    let eventId: string | undefined = hint && hint.event_id;
+    this._processing = true;
+
+    this._processEvent(event, hint, scope)
+      .then(finalEvent => {
+        // We need to check for finalEvent in case beforeSend returned null
+        eventId = finalEvent && finalEvent.event_id;
+        // We do this here to not parse any requests if they are outgoing to Sentry
+        // We log breadcrumbs if the integration has them enabled
+        if (finalEvent) {
+          const integration = this.getIntegration(Breadcrumbs);
+          if (integration) {
+            integration.addSentryBreadcrumb(finalEvent);
+          }
+        }
+        this._processing = false;
+      })
+      .then(null, reason => {
+        logger.error(reason);
+        this._processing = false;
+      });
+
+    return eventId;
+  }
+
   /**
    * Show a report dialog to the user to send feedback to a specific event.
    *
diff --git a/packages/browser/src/integrations/breadcrumbs.ts b/packages/browser/src/integrations/breadcrumbs.ts
@@ -1,17 +1,14 @@
-import { API, getCurrentHub } from '@sentry/core';
-import { Integration, Severity } from '@sentry/types';
+import { getCurrentHub } from '@sentry/core';
+import { Event, Integration, Severity } from '@sentry/types';
 import {
   addInstrumentationHandler,
   getEventDescription,
   getGlobalObject,
   htmlTreeAsString,
-  logger,
   parseUrl,
   safeJoin,
 } from '@sentry/utils';
 
-import { BrowserClient } from '../client';
-
 /**
  * @hidden
  */
@@ -67,6 +64,26 @@ export class Breadcrumbs implements Integration {
     };
   }
 
+  /**
+   * Create a breadcrumb of `sentry` from the events themselves
+   */
+  public addSentryBreadcrumb(event: Event): void {
+    if (!this._options.sentry) {
+      return;
+    }
+    getCurrentHub().addBreadcrumb(
+      {
+        category: `sentry.${event.type === 'transaction' ? 'transaction' : 'event'}`,
+        event_id: event.event_id,
+        level: event.level,
+        message: getEventDescription(event),
+      },
+      {
+        event,
+      },
+    );
+  }
+
   /**
    * Creates breadcrumbs from console API calls
    */
@@ -151,11 +168,6 @@ export class Breadcrumbs implements Integration {
 
       return;
     }
-
-    // We only capture issued sentry requests
-    if (this._options.sentry && handlerData.xhr.__sentry_own_request__) {
-      addSentryBreadcrumb(handlerData.args[0]);
-    }
   }
 
   /**
@@ -167,24 +179,6 @@ export class Breadcrumbs implements Integration {
       return;
     }
 
-    const client = getCurrentHub().getClient<BrowserClient>();
-    const dsn = client && client.getDsn();
-    if (this._options.sentry && dsn) {
-      const filterUrl = new API(dsn).getBaseApiEndpoint();
-      // if Sentry key appears in URL, don't capture it as a request
-      // but rather as our own 'sentry' type breadcrumb
-      if (
-        filterUrl &&
-        handlerData.fetchData.url.indexOf(filterUrl) !== -1 &&
-        handlerData.fetchData.method === 'POST' &&
-        handlerData.args[1] &&
-        handlerData.args[1].body
-      ) {
-        addSentryBreadcrumb(handlerData.args[1].body);
-        return;
-      }
-    }
-
     if (handlerData.error) {
       getCurrentHub().addBreadcrumb(
         {
@@ -306,26 +300,3 @@ export class Breadcrumbs implements Integration {
     }
   }
 }
-
-/**
- * Create a breadcrumb of `sentry` from the events themselves
- */
-function addSentryBreadcrumb(serializedData: string): void {
-  // There's always something that can go wrong with deserialization...
-  try {
-    const event = JSON.parse(serializedData);
-    getCurrentHub().addBreadcrumb(
-      {
-        category: `sentry.${event.type === 'transaction' ? 'transaction' : 'event'}`,
-        event_id: event.event_id,
-        level: event.level || Severity.fromString('error'),
-        message: getEventDescription(event),
-      },
-      {
-        event,
-      },
-    );
-  } catch (_oO) {
-    logger.error('Error while adding sentry type breadcrumb');
-  }
-}
