fix: Domain require

Author: HazAT

packages/hub/src/hub.ts

@@ -454,8 +454,14 @@ export function getCurrentHub(): Hub {
  */
 function getHubFromActiveDomain(registry: Carrier): Hub {
   try {
-    const req = require;
-    const domain = req('domain');
+    const property = 'domain';
+    const carrier = getMainCarrier();
+    const sentry = carrier.__SENTRY__;
+    // tslint:disable-next-line: strict-type-predicates
+    if (!sentry || !sentry.extensions || !sentry.extensions[property]) {
+      return getHubFromCarrier(registry);
+    }
+    const domain = sentry.extensions[property] as any;
     const activeDomain = domain.active;
 
     // If there no active domain, just return global hub

packages/node/src/index.ts

@@ -39,6 +39,8 @@ export { defaultIntegrations, init, lastEventId, flush, close } from './sdk';
 export { SDK_NAME, SDK_VERSION } from './version';
 
 import { Integrations as CoreIntegrations } from '@sentry/core';
+import { getMainCarrier } from '@sentry/hub';
+import * as domain from 'domain';
 
 import * as Handlers from './handlers';
 import * as NodeIntegrations from './integrations';
@@ -50,3 +52,14 @@ const INTEGRATIONS = {
 };
 
 export { INTEGRATIONS as Integrations, Transports, Handlers };
+
+// We need to patch domain on the global __SENTRY__ object to make it work for node
+// if we don't do this, browser bundlers will have troubles resolving require('domain')
+const carrier = getMainCarrier();
+if (carrier.__SENTRY__) {
+  carrier.__SENTRY__.extensions = carrier.__SENTRY__.extensions || {};
+  if (!carrier.__SENTRY__.extensions.domain) {
+    // @ts-ignore
+    carrier.__SENTRY__.extensions.domain = domain;
+  }
+}

packages/node/test/domain.test.ts

@@ -1,6 +1,10 @@
 import { getCurrentHub, Hub } from '@sentry/core';
 import * as domain from 'domain';
 
+import * as Sentry from '../src';
+
+console.log(Sentry.SDK_NAME);
+
 describe('domains', () => {
   test('without domain', () => {
     expect(domain.active).toBeFalsy();