getsentry · Sep 21, 2021 · Sep 22, 2021 · Sep 23, 2021 · Sep 27, 2021 · Sep 27, 2021
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,17 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
+## 6.13.3
+
+- feat(nextjs): Add ability for integration tests to use linked `@sentry/xxxx` packages (#4019)
+- feat(nextjs): Support `distDir` Next.js option (#3990)
+- fix(tracing): Call hasTracingEnabled with correct options when invoking startTransaction (#4020)
+- ref(browser): Refactor sending client reports w. fetch fallback (#4008)
+- ref(core): Make getTransport method on client optional (#4013)
+- ref(ember): Update htmlbars dependency (#4026)
+- ref(integrations): Minor simplification of ExtraErrorData code (#4024)
+- ref(react): Rely on error.cause to link ErrorBoundary errors (#4005)
+
 ## 6.13.2
 
 - fix(browser): Use getGlobalObject for document check (#3996)

diff --git a/lerna.json b/lerna.json
@@ -1,6 +1,6 @@
 {
   "lerna": "3.4.0",
-  "version": "6.13.2",
+  "version": "6.13.3",
   "packages": "packages/*",
   "npmClient": "yarn",
   "useWorkspaces": true

diff --git a/packages/angular/package.json b/packages/angular/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@sentry/angular",
-  "version": "6.13.2",
+  "version": "6.13.3",
   "description": "Official Sentry SDK for Angular",
   "repository": "git://github.com/getsentry/sentry-javascript.git",
   "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/angular",
@@ -21,17 +21,17 @@
     "@angular/router": "10.x || 11.x || 12.x"
   },
   "dependencies": {
-    "@sentry/browser": "6.13.2",
-    "@sentry/types": "6.13.2",
-    "@sentry/utils": "6.13.2",
+    "@sentry/browser": "6.13.3",
+    "@sentry/types": "6.13.3",
+    "@sentry/utils": "6.13.3",
     "rxjs": "^6.6.0",
     "tslib": "^1.9.3"
   },
   "devDependencies": {
     "@angular/common": "^10.0.3",
     "@angular/core": "^10.0.3",
     "@angular/router": "^10.0.3",
-    "@sentry-internal/eslint-config-sdk": "6.13.2",
+    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "npm-run-all": "^4.1.2",
     "prettier": "1.19.0",
     "rimraf": "^2.6.3",

diff --git a/packages/browser/package.json b/packages/browser/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@sentry/browser",
-  "version": "6.13.2",
+  "version": "6.13.3",
   "description": "Official Sentry SDK for browsers",
   "repository": "git://github.com/getsentry/sentry-javascript.git",
   "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/browser",
@@ -16,13 +16,13 @@
     "access": "public"
   },
   "dependencies": {
-    "@sentry/core": "6.13.2",
-    "@sentry/types": "6.13.2",
-    "@sentry/utils": "6.13.2",
+    "@sentry/core": "6.13.3",
+    "@sentry/types": "6.13.3",
+    "@sentry/utils": "6.13.3",
     "tslib": "^1.9.3"
   },
   "devDependencies": {
-    "@sentry-internal/eslint-config-sdk": "6.13.2",
+    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "@types/eslint": "^7.2.0",
     "@types/md5": "2.1.33",
     "btoa": "^1.2.1",

diff --git a/packages/browser/src/transports/base.ts b/packages/browser/src/transports/base.ts
@@ -17,6 +17,8 @@ import {
   SentryError,
 } from '@sentry/utils';
 
+import { sendReport } from './utils';
+
 const CATEGORY_MAPPING: {
   [key in SentryRequestType]: string;
 } = {
@@ -99,11 +101,6 @@ export abstract class BaseTransport implements Transport {
       return;
     }
 
-    if (!global.navigator || typeof global.navigator.sendBeacon !== 'function') {
-      logger.warn('Beacon API not available, skipping sending outcomes.');
-      return;
-    }
-
     const outcomes = this._outcomes;
     this._outcomes = {};
 
@@ -134,7 +131,11 @@ export abstract class BaseTransport implements Transport {
     });
     const envelope = `${envelopeHeader}\n${itemHeaders}\n${item}`;
 
-    global.navigator.sendBeacon(url, envelope);
+    try {
+      sendReport(url, envelope);
+    } catch (e) {
+      logger.error(e);
+    }
   }
 
   /**

diff --git a/packages/browser/src/transports/fetch.ts b/packages/browser/src/transports/fetch.ts
@@ -1,85 +1,9 @@
 import { eventToSentryRequest, sessionToSentryRequest } from '@sentry/core';
 import { Event, Outcome, Response, SentryRequest, Session, TransportOptions } from '@sentry/types';
-import {
-  getGlobalObject,
-  isNativeFetch,
-  logger,
-  SentryError,
-  supportsReferrerPolicy,
-  SyncPromise,
-} from '@sentry/utils';
+import { SentryError, supportsReferrerPolicy, SyncPromise } from '@sentry/utils';
 
 import { BaseTransport } from './base';
-
-type FetchImpl = typeof fetch;
-
-/**
- * A special usecase for incorrectly wrapped Fetch APIs in conjunction with ad-blockers.
- * Whenever someone wraps the Fetch API and returns the wrong promise chain,
- * this chain becomes orphaned and there is no possible way to capture it's rejections
- * other than allowing it bubble up to this very handler. eg.
- *
- * const f = window.fetch;
- * window.fetch = function () {
- *   const p = f.apply(this, arguments);
- *
- *   p.then(function() {
- *     console.log('hi.');
- *   });
- *
- *   return p;
- * }
- *
- * `p.then(function () { ... })` is producing a completely separate promise chain,
- * however, what's returned is `p` - the result of original `fetch` call.
- *
- * This mean, that whenever we use the Fetch API to send our own requests, _and_
- * some ad-blocker blocks it, this orphaned chain will _always_ reject,
- * effectively causing another event to be captured.
- * This makes a whole process become an infinite loop, which we need to somehow
- * deal with, and break it in one way or another.
- *
- * To deal with this issue, we are making sure that we _always_ use the real
- * browser Fetch API, instead of relying on what `window.fetch` exposes.
- * The only downside to this would be missing our own requests as breadcrumbs,
- * but because we are already not doing this, it should be just fine.
- *
- * Possible failed fetch error messages per-browser:
- *
- * Chrome:  Failed to fetch
- * Edge:    Failed to Fetch
- * Firefox: NetworkError when attempting to fetch resource
- * Safari:  resource blocked by content blocker
- */
-function getNativeFetchImplementation(): FetchImpl {
-  /* eslint-disable @typescript-eslint/unbound-method */
-
-  // Fast path to avoid DOM I/O
-  const global = getGlobalObject<Window>();
-  if (isNativeFetch(global.fetch)) {
-    return global.fetch.bind(global);
-  }
-
-  const document = global.document;
-  let fetchImpl = global.fetch;
-  // eslint-disable-next-line deprecation/deprecation
-  if (typeof document?.createElement === `function`) {
-    try {
-      const sandbox = document.createElement('iframe');
-      sandbox.hidden = true;
-      document.head.appendChild(sandbox);
-      if (sandbox.contentWindow?.fetch) {
-        fetchImpl = sandbox.contentWindow.fetch;
-      }
-      document.head.removeChild(sandbox);
-    } catch (e) {
-      logger.warn('Could not create sandbox iframe for pure fetch check, bailing to window.fetch: ', e);
-    }
-  }
-
-  return fetchImpl.bind(global);
-  /* eslint-enable @typescript-eslint/unbound-method */
-}
+import { FetchImpl, getNativeFetchImplementation } from './utils';
 
 /** `fetch` based transport */
 export class FetchTransport extends BaseTransport {

diff --git a/packages/browser/src/transports/utils.ts b/packages/browser/src/transports/utils.ts
@@ -0,0 +1,106 @@
+import { forget, getGlobalObject, isNativeFetch, logger, supportsFetch } from '@sentry/utils';
+
+const global = getGlobalObject<Window>();
+let cachedFetchImpl: FetchImpl;
+
+export type FetchImpl = typeof fetch;
+
+/**
+ * A special usecase for incorrectly wrapped Fetch APIs in conjunction with ad-blockers.
+ * Whenever someone wraps the Fetch API and returns the wrong promise chain,
+ * this chain becomes orphaned and there is no possible way to capture it's rejections
+ * other than allowing it bubble up to this very handler. eg.
+ *
+ * const f = window.fetch;
+ * window.fetch = function () {
+ *   const p = f.apply(this, arguments);
+ *
+ *   p.then(function() {
+ *     console.log('hi.');
+ *   });
+ *
+ *   return p;
+ * }
+ *
+ * `p.then(function () { ... })` is producing a completely separate promise chain,
+ * however, what's returned is `p` - the result of original `fetch` call.
+ *
+ * This mean, that whenever we use the Fetch API to send our own requests, _and_
+ * some ad-blocker blocks it, this orphaned chain will _always_ reject,
+ * effectively causing another event to be captured.
+ * This makes a whole process become an infinite loop, which we need to somehow
+ * deal with, and break it in one way or another.
+ *
+ * To deal with this issue, we are making sure that we _always_ use the real
+ * browser Fetch API, instead of relying on what `window.fetch` exposes.
+ * The only downside to this would be missing our own requests as breadcrumbs,
+ * but because we are already not doing this, it should be just fine.
+ *
+ * Possible failed fetch error messages per-browser:
+ *
+ * Chrome:  Failed to fetch
+ * Edge:    Failed to Fetch
+ * Firefox: NetworkError when attempting to fetch resource
+ * Safari:  resource blocked by content blocker
+ */
+export function getNativeFetchImplementation(): FetchImpl {
+  if (cachedFetchImpl) {
+    return cachedFetchImpl;
+  }
+
+  /* eslint-disable @typescript-eslint/unbound-method */
+
+  // Fast path to avoid DOM I/O
+  if (isNativeFetch(global.fetch)) {
+    return (cachedFetchImpl = global.fetch.bind(global));
+  }
+
+  const document = global.document;
+  let fetchImpl = global.fetch;
+  // eslint-disable-next-line deprecation/deprecation
+  if (typeof document?.createElement === `function`) {
+    try {
+      const sandbox = document.createElement('iframe');
+      sandbox.hidden = true;
+      document.head.appendChild(sandbox);
+      if (sandbox.contentWindow?.fetch) {
+        fetchImpl = sandbox.contentWindow.fetch;
+      }
+      document.head.removeChild(sandbox);
+    } catch (e) {
+      logger.warn('Could not create sandbox iframe for pure fetch check, bailing to window.fetch: ', e);
+    }
+  }
+
+  return (cachedFetchImpl = fetchImpl.bind(global));
+  /* eslint-enable @typescript-eslint/unbound-method */
+}
+
+/**
+ * Sends sdk client report using sendBeacon or fetch as a fallback if available
+ *
+ * @param url report endpoint
+ * @param body report payload
+ */
+export function sendReport(url: string, body: string): void {
+  const isRealNavigator = Object.prototype.toString.call(global && global.navigator) === '[object Navigator]';
+  const hasSendBeacon = isRealNavigator && typeof global.navigator.sendBeacon === 'function';
+
+  if (hasSendBeacon) {
+    // Prevent illegal invocations - https://xgwang.me/posts/you-may-not-know-beacon/#it-may-throw-error%2C-be-sure-to-catch
+    const sendBeacon = global.navigator.sendBeacon.bind(global.navigator);
+    return sendBeacon(url, body);
+  }
+
+  if (supportsFetch()) {
+    const fetch = getNativeFetchImplementation();
+    return forget(
+      fetch(url, {
+        body,
+        method: 'POST',
+        credentials: 'omit',
+        keepalive: true,
+      }),
+    );
+  }
+}
diff --git a/packages/core/package.json b/packages/core/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@sentry/core",
-  "version": "6.13.2",
+  "version": "6.13.3",
   "description": "Base implementation for all Sentry JavaScript SDKs",
   "repository": "git://github.com/getsentry/sentry-javascript.git",
   "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/core",
@@ -16,14 +16,14 @@
     "access": "public"
   },
   "dependencies": {
-    "@sentry/hub": "6.13.2",
-    "@sentry/minimal": "6.13.2",
-    "@sentry/types": "6.13.2",
-    "@sentry/utils": "6.13.2",
+    "@sentry/hub": "6.13.3",
+    "@sentry/minimal": "6.13.3",
+    "@sentry/types": "6.13.3",
+    "@sentry/utils": "6.13.3",
     "tslib": "^1.9.3"
   },
   "devDependencies": {
-    "@sentry-internal/eslint-config-sdk": "6.13.2",
+    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "jest": "^24.7.1",
     "npm-run-all": "^4.1.2",
     "prettier": "1.19.0",

diff --git a/packages/core/src/version.ts b/packages/core/src/version.ts
@@ -1 +1 @@
-export const SDK_VERSION = '6.13.2';
+export const SDK_VERSION = '6.13.3';
diff --git a/packages/ember/package.json b/packages/ember/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@sentry/ember",
-  "version": "6.13.2",
+  "version": "6.13.3",
   "description": "Official Sentry SDK for Ember.js",
   "repository": "git://github.com/getsentry/sentry-javascript.git",
   "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/ember",
@@ -32,21 +32,21 @@
   },
   "dependencies": {
     "@embroider/macros": "~0.37.0",
-    "@sentry/browser": "6.13.2",
-    "@sentry/tracing": "6.13.2",
-    "@sentry/types": "6.13.2",
-    "@sentry/utils": "6.13.2",
+    "@sentry/browser": "6.13.3",
+    "@sentry/tracing": "6.13.3",
+    "@sentry/types": "6.13.3",
+    "@sentry/utils": "6.13.3",
     "ember-auto-import": "~1.6.0",
     "ember-cli-babel": "~7.26.6",
-    "ember-cli-htmlbars": "~5.1.2",
+    "ember-cli-htmlbars": "^5.7.1",
     "ember-cli-typescript": "~3.1.4"
   },
   "devDependencies": {
     "@ember/optional-features": "~1.3.0",
     "@embroider/test-setup": "~0.37.0",
     "@glimmer/component": "~1.0.0",
     "@glimmer/tracking": "~1.0.0",
-    "@sentry-internal/eslint-config-sdk": "6.13.2",
+    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "@types/ember": "~3.16.5",
     "@types/ember-qunit": "~3.4.9",
     "@types/ember__test-helpers": "~1.7.0",

diff --git a/packages/eslint-config-sdk/package.json b/packages/eslint-config-sdk/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@sentry-internal/eslint-config-sdk",
-  "version": "6.13.2",
+  "version": "6.13.3",
   "description": "Official Sentry SDK eslint config",
   "repository": "git://github.com/getsentry/sentry-javascript.git",
   "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/eslint-config-sdk",
@@ -19,8 +19,8 @@
     "access": "public"
   },
   "dependencies": {
-    "@sentry-internal/eslint-plugin-sdk": "6.13.2",
-    "@sentry-internal/typescript": "6.13.2",
+    "@sentry-internal/eslint-plugin-sdk": "6.13.3",
+    "@sentry-internal/typescript": "6.13.3",
     "@typescript-eslint/eslint-plugin": "^3.9.0",
     "@typescript-eslint/parser": "^3.9.0",
     "eslint-config-prettier": "^6.11.0",
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		export const SDK_VERSION = '6.13.2';
		export const SDK_VERSION = '6.13.3';