feat: Add getGlobalObject method to utils

Author: kamilogorek

packages/browser/src/integrations/onunhandledrejection.ts

@@ -1,13 +1,8 @@
 import { captureException } from '@sentry/minimal';
 import { Integration } from '@sentry/types';
+import { getGlobalObject } from '@sentry/utils';
 
-/** onunhandledrejection is not standardized, thus not available on Window type */
-interface PromisifiedWindow extends Window {
-  onunhandledrejection?(event: PromiseRejectionEvent): void;
-}
-
-/** TODO: Change to safe window access, window||global||self||{} */
-const _window: PromisifiedWindow = window;
+const global: any = getGlobalObject();
 
 /** Global Promise Rejection handler */
 export class OnUnhandledRejection implements Integration {
@@ -25,8 +20,10 @@ export class OnUnhandledRejection implements Integration {
    * @inheritDoc
    */
   public install(): void {
-    _window.addEventListener('unhandledrejection', this.handler.bind(
-      this,
-    ) as EventListener);
+    if (global.addEventListener) {
+      global.addEventListener('unhandledrejection', this.handler.bind(
+        this,
+      ) as EventListener);
+    }
   }
 }

packages/browser/src/transports/fetch.ts

@@ -1,7 +1,13 @@
 import { SentryEvent } from '@sentry/types';
-import { serialize, supportsReferrerPolicy } from '@sentry/utils';
+import {
+  getGlobalObject,
+  serialize,
+  supportsReferrerPolicy,
+} from '@sentry/utils';
 import { BaseTransport } from './base';
 
+const global: any = getGlobalObject();
+
 /** `fetch` based transport */
 export class FetchTransport extends BaseTransport {
   /**
@@ -21,7 +27,6 @@ export class FetchTransport extends BaseTransport {
         : '') as ReferrerPolicy,
     };
 
-    // TODO: Safe _window access
-    return window.fetch(this.url, defaultOptions);
+    return global.fetch(this.url, defaultOptions);
   }
 }

packages/utils/src/index.ts

@@ -11,3 +11,4 @@ export {
   supportsFetch,
   supportsReferrerPolicy,
 } from './supports';
+export { getGlobalObject } from './misc';

packages/utils/src/misc.ts

@@ -0,0 +1,14 @@
+/**
+ * Safely get global scope object
+ *
+ * @returns Global scope object
+ */
+// tslint:disable:strict-type-predicates
+export function getGlobalObject(): Window | NodeJS.Global | {} {
+  return typeof window !== 'undefined'
+    ? window
+    : typeof global !== 'undefined'
+      ? global
+      : typeof self !== 'undefined' ? self : {};
+}
+// tslint:enable:strict-type-predicates

packages/utils/src/string.ts

@@ -1,3 +1,5 @@
+import { getGlobalObject } from './misc';
+
 /**
  * Encodes given object into url-friendly format
  *
@@ -26,8 +28,8 @@ interface MsCryptoWindow extends Window {
  * @returns string Generated UUID4.
  */
 export function uuid4(): string {
-  // TODO: Safe window access
-  const crypto = window.crypto || (window as MsCryptoWindow).msCrypto;
+  const global = getGlobalObject() as MsCryptoWindow;
+  const crypto = global.crypto || global.msCrypto;
 
   if (!(crypto === void 0) && crypto.getRandomValues) {
     // Use window.crypto API if available

packages/utils/src/supports.ts

@@ -1,3 +1,5 @@
+import { getGlobalObject } from './misc';
+
 /**
  * Tells whether current environment supports ErrorEvent objects
  * {@link supportsErrorEvent}.
@@ -57,8 +59,7 @@ export function supportsDOMException(): boolean {
  * @returns Answer to the given question.
  */
 export function supportsFetch(): boolean {
-  // TODO: Safe window access
-  if (!('fetch' in window)) {
+  if (!('fetch' in getGlobalObject())) {
     return false;
   }