fix: Transports, Add additional info in hint for global errros

Author: HazAT

packages/browser/src/integrations/globalhandlers.ts

@@ -1,5 +1,5 @@
 import { logger } from '@sentry/core';
-import { captureEvent } from '@sentry/minimal';
+import { getCurrentHub } from '@sentry/hub';
 import { Integration, SentryEvent } from '@sentry/types';
 import { eventFromStacktrace } from '../parsers';
 import {
@@ -29,7 +29,7 @@ export class GlobalHandlers implements Integration {
    * @inheritDoc
    */
   public install(): void {
-    subscribe((stack: TraceKitStackTrace) => {
+    subscribe((stack: TraceKitStackTrace, _: boolean, error: Error) => {
       // TODO: use stack.context to get a valuable information from TraceKit, eg.
       // [
       //   0: "  })"
@@ -47,7 +47,7 @@ export class GlobalHandlers implements Integration {
       if (shouldIgnoreOnError()) {
         return;
       }
-      captureEvent(this.eventFromGlobalHandler(stack));
+      getCurrentHub().captureEvent(this.eventFromGlobalHandler(stack), { originalException: error, data: stack });
     });
 
     if (this.options.onerror) {

packages/browser/src/integrations/helpers.ts

@@ -79,7 +79,7 @@ export function wrap(
           });
         });
 
-        getCurrentHub().captureException(ex);
+        getCurrentHub().captureException(ex, { originalException: ex });
       });
 
       throw ex;

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

@@ -41,7 +41,6 @@ describe('FetchTransport', () => {
         expect(
           fetch.calledWith(transportUrl, {
             body: JSON.stringify(payload),
-            keepalive: true,
             method: 'POST',
             referrerPolicy: 'origin',
           }),
@@ -60,7 +59,6 @@ describe('FetchTransport', () => {
         expect(
           fetch.calledWith(transportUrl, {
             body: JSON.stringify(payload),
-            keepalive: true,
             method: 'POST',
             referrerPolicy: 'origin',
           }),

packages/hub/src/hub.ts

@@ -201,10 +201,12 @@ export class Hub {
    * Captures a manually created event and sends it to Sentry.
    *
    * @param event The event to send to Sentry.
+   * @param hint May contain additional informartion about the original exception.
    */
-  public captureEvent(event: SentryEvent): string {
+  public captureEvent(event: SentryEvent, hint?: SentryEventHint): string {
     const eventId = (this._lastEventId = uuid4());
     this.invokeClientAsync('captureEvent', event, {
+      ...hint,
       event_id: eventId,
     });
     return eventId;

packages/node/src/handlers.ts

@@ -192,7 +192,7 @@ export function errorHandler(): (
       next(error);
       return;
     }
-    getHubFromCarrier(req).captureException(error);
+    getHubFromCarrier(req).captureException(error, { originalException: error });
     next(error);
   };
 }
@@ -229,7 +229,7 @@ export function makeErrorHandler(
           }));
         });
 
-        getCurrentHub().captureException(error);
+        getCurrentHub().captureException(error, { originalException: error });
 
         if (!calledFatalError) {
           calledFatalError = true;

packages/node/src/integrations/onunhandledrejection.ts

@@ -39,7 +39,7 @@ export class OnUnhandledRejection implements Integration {
         }
         scope.setExtra('unhandledPromiseRejection', true);
       });
-      getCurrentHub().captureException(reason);
+      getCurrentHub().captureException(reason, { originalException: promise });
     });
   }
 }