fix: On fatal error callback (#1426)

* fix: On fatal error callback

* feat: Export default integrations

* fix: Use defaultIntegrations

Author: HazAT

packages/browser/src/index.ts

@@ -25,7 +25,7 @@ export { getHubFromCarrier, getDefaultHub, Hub, Scope } from '@sentry/hub';
 
 export { BrowserBackend, BrowserOptions } from './backend';
 export { BrowserClient } from './client';
-export { init } from './sdk';
+export { defaultIntegrations, init } from './sdk';
 
 import * as Integrations from './integrations';
 import * as Transports from './transports';

packages/browser/src/sdk.ts

@@ -9,6 +9,14 @@ import {
   TryCatch,
 } from './integrations';
 
+export const defaultIntegrations = [
+  new Breadcrumbs(),
+  new FunctionToString(),
+  new OnError(),
+  new OnUnhandledRejection(),
+  new TryCatch(),
+];
+
 /**
  * The Sentry Browser SDK Client.
  *
@@ -53,11 +61,5 @@ import {
  * @see BrowserOptions for documentation on configuration options.
  */
 export function init(options: BrowserOptions): void {
-  initAndBind(BrowserClient, options, [
-    new OnError(),
-    new OnUnhandledRejection(),
-    new FunctionToString(),
-    new TryCatch(),
-    new Breadcrumbs(),
-  ]);
+  initAndBind(BrowserClient, options, defaultIntegrations);
 }

packages/node/src/backend.ts

@@ -80,6 +80,16 @@ export class NodeBackend implements Backend {
       if (callback && (callback as FunctionExt).__SENTRY_CAPTURE__) {
         callback(normalizeEvent(event));
       } else {
+        // This "if" needs to be here in order for raven-node to propergate
+        // internal callbacks like in makeErrorHandler -> captureException
+        // correctly. Also the setTimeout is a dirty hack because in case of a
+        // fatal error, raven-node exits the process and our consturct of
+        // hub -> client doesn't have enough time to send the event.
+        if (callback) {
+          setTimeout(() => {
+            callback(event);
+          }, 1000);
+        }
         captureEvent(normalizeEvent(event));
       }
     };

packages/node/src/index.ts

@@ -26,7 +26,7 @@ export { getHubFromCarrier, Hub, Scope } from '@sentry/hub';
 export { getDefaultHub } from './hub';
 export { NodeBackend, NodeOptions } from './backend';
 export { NodeClient } from './client';
-export { init } from './sdk';
+export { defaultIntegrations, init } from './sdk';
 
 import * as Integrations from './integrations';
 import * as Transports from './transports';

packages/node/src/sdk.ts

@@ -8,6 +8,13 @@ import {
   OnUnhandledRejection,
 } from './integrations';
 
+export const defaultIntegrations = [
+  new Console(),
+  new Http(),
+  new OnUncaughtException(),
+  new OnUnhandledRejection(),
+];
+
 /**
  * The Sentry Node SDK Client.
  *
@@ -53,10 +60,5 @@ import {
  * @see NodeOptions for documentation on configuration options.
  */
 export function init(options: NodeOptions): void {
-  initAndBind(NodeClient, options, [
-    new OnUncaughtException(),
-    new OnUnhandledRejection(),
-    new Console(),
-    new Http(),
-  ]);
+  initAndBind(NodeClient, options, defaultIntegrations);
 }