Update to Sentry v7.0.0 (Fixes mozilla#10473)

alexgibson · May 31, 2022 · c477f21 · c477f21
1 parent dba56e4
commit c477f21
Show file tree

Hide file tree

Showing 5 changed files with 82 additions and 111 deletions.
diff --git a/media/js/base/sentry.es6.js b/media/js/base/sentry.es6.js
@@ -4,7 +4,15 @@
  * file, You can obtain one at https://mozilla.org/MPL/2.0/.
  */
 
-import { init } from '@sentry/browser';
+import {
+    BrowserClient,
+    Dedupe,
+    GlobalHandlers,
+    HttpContext,
+    defaultStackParser,
+    getCurrentHub,
+    makeFetchTransport
+} from '@sentry/browser';
 
 // Respect Do Not Track
 if (
@@ -16,32 +24,13 @@ if (
         .getElementsByTagName('html')[0]
         .getAttribute('data-sentry-dsn');
 
-    // Configure Sentry SDK
-    if (sentryDsn) {
-        init({
-            dsn: sentryDsn,
-            sampleRate: 0.1,
-            ignoreErrors: [
-                "SecurityError: Failed to execute 'open' on 'XMLHttpRequest'", // issue 10683
-                'NetworkError when attempting to fetch resource', // issue 10683
-                'Non-Error promise rejection captured with value' // issue 10380
-            ],
-            allowUrls: ['/media/js/', 'https://cdn-3.convertexperiments.com/'],
-            beforeSend(event) {
-                try {
-                    // https://github.com/getsentry/sentry-javascript/issues/3147
-                    if (
-                        event.exception.values[0].stacktrace.frames[0]
-                            .filename === '<anonymous>'
-                    ) {
-                        return null;
-                    }
-                } catch (e) {
-                    // do nothing.
-                }
+    const client = new BrowserClient({
+        dsn: sentryDsn,
+        sampleRate: 0.1,
+        transport: makeFetchTransport,
+        stackParser: defaultStackParser,
+        integrations: [new Dedupe(), new GlobalHandlers(), new HttpContext()]
+    });
 
-                return event;
-            }
-        });
-    }
+    getCurrentHub().bindClient(client);
 }
diff --git a/media/js/mozorg/home/youtube.js b/media/js/mozorg/home/youtube.js
@@ -145,4 +145,6 @@ window.onYouTubeIframeAPIReady = function () {
     Mozilla.homePageVideoPlay = playVideo;
 
     init();
+
+    myUndefinedFunction(); // eslint-disable-line no-undef
 })();
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -8,7 +8,7 @@
     "@babel/preset-env": "^7.16.11",
     "@mozilla-protocol/core": "16.0.0",
     "@mozilla/glean": "^1.0.0",
-    "@sentry/browser": "^6.19.0",
+    "@sentry/browser": "^7.0.0",
     "babel-loader": "^8.2.4",
     "clean-webpack-plugin": "^4.0.0",
     "copy-webpack-plugin": "^10.2.4",

diff --git a/webpack.config.js b/webpack.config.js
@@ -13,6 +13,7 @@ const Dotenv = require('dotenv-webpack');
 const MiniCssExtractPlugin = require('mini-css-extract-plugin');
 const path = require('path');
 const staticBundles = require('./media/static-bundles.json');
+const webpack = require('webpack');
 
 function resolveBundles(fileList) {
     return fileList.map((f) => {
@@ -99,7 +100,11 @@ const jsConfig = {
                 }
             ]
         }),
-        new Dotenv()
+        new Dotenv(),
+        new webpack.DefinePlugin({
+            __SENTRY_DEBUG__: false,
+            __SENTRY_TRACING__: false
+        })
     ]
 };