feat: Emit warning when some browser APIs are not in place

kamilogorek · kamilogorek · commit 3e491126a812 · 2019-07-05T14:49:20.000+02:00
diff --git a/packages/browser/src/helpers.ts b/packages/browser/src/helpers.ts
@@ -7,6 +7,39 @@ let keypressTimeout: number | undefined;
 let lastCapturedEvent: Event | undefined;
 let ignoreOnError: number = 0;
 
+/**
+ * @hidden
+ */
+export function verifyRequiredBrowserAPIs(): void {
+  // tslint:disable:strict-type-predicates no-unbound-method
+  const missingAPIs = [];
+
+  if (typeof Promise === 'undefined') {
+    missingAPIs.push('Promise');
+  }
+
+  if (typeof Object.assign === 'undefined') {
+    missingAPIs.push('Object.assign');
+  }
+
+  if (typeof Number.isNaN === 'undefined') {
+    missingAPIs.push('Number.isNaN');
+  }
+
+  if (typeof String.prototype.includes === 'undefined') {
+    missingAPIs.push('String.includes');
+  }
+
+  if (missingAPIs.length > 0) {
+    // tslint:disable-next-line:no-console
+    console.warn(
+      'Your current browser is missing following APIs:',
+      missingAPIs.join(', '),
+      'Sentry SDK may not work as expected. Please provide necessary polyfills. https://docs.sentry.io/platforms/javascript/#browser-table',
+    );
+  }
+}
+
 /**
  * @hidden
  */
diff --git a/packages/browser/src/sdk.ts b/packages/browser/src/sdk.ts
@@ -3,7 +3,7 @@ import { getGlobalObject } from '@sentry/utils';
 
 import { BrowserOptions } from './backend';
 import { BrowserClient, ReportDialogOptions } from './client';
-import { wrap as internalWrap } from './helpers';
+import { verifyRequiredBrowserAPIs, wrap as internalWrap } from './helpers';
 import { Breadcrumbs, GlobalHandlers, LinkedErrors, TryCatch, UserAgent } from './integrations';
 
 export const defaultIntegrations = [
@@ -74,6 +74,12 @@ export const defaultIntegrations = [
  * @see {@link BrowserOptions} for documentation on configuration options.
  */
 export function init(options: BrowserOptions = {}): void {
+  try {
+    verifyRequiredBrowserAPIs();
+  } catch (_oO) {
+    // juuust in case
+  }
+
   if (options.defaultIntegrations === undefined) {
     options.defaultIntegrations = defaultIntegrations;
   }
