Don't send errors to sentry if users have not opted-in to participate in metametrics

Author: danjm

app/scripts/background.js

@@ -383,7 +383,9 @@ function setupController(initState, initLangCode, remoteSourcePort) {
         // log error so we dont break the pipeline
         if (!dataPersistenceFailing) {
           dataPersistenceFailing = true;
-          captureException(err);
+          if (state.data.metamask.participateInMetaMetrics) {
+            captureException(err);
+          }
         }
         log.error('error setting state in local store:', err);
       }

app/scripts/lib/setupSentry.js

@@ -103,7 +103,17 @@ export default function setupSentry({ release, getState }) {
     environment,
     integrations: [new Dedupe(), new ExtraErrorData()],
     release,
-    beforeSend: (report) => rewriteReport(report),
+    beforeSend: (report) => {
+      if (getState) {
+        const appState = getState();
+        if (!appState?.store?.metamask?.participateInMetaMetrics) {
+          return null;
+        }
+      } else {
+        return null;
+      }
+      return rewriteReport(report);
+    },
   });
 
   function rewriteReport(report) {

app/scripts/metamask-controller.js

@@ -341,6 +341,12 @@ export default class MetamaskController extends EventEmitter {
         },
       ));
 
+    this.captureException = (...args) => {
+      return this.getState()?.metamask?.participateInMetaMetrics
+        ? captureException(...args)
+        : undefined;
+    };
+
     this.metaMetricsController = new MetaMetricsController({
       segment,
       preferencesStore: this.preferencesController.store,
@@ -357,7 +363,7 @@ export default class MetamaskController extends EventEmitter {
       version: this.platform.getVersion(),
       environment: process.env.METAMASK_ENVIRONMENT,
       initState: initState.MetaMetricsController,
-      captureException,
+      captureException: this.captureException.bind(this),
     });
 
     this.on('update', (update) => {
@@ -595,7 +601,7 @@ export default class MetamaskController extends EventEmitter {
               this.accountTracker.store.getState().accounts || {},
             ).length;
 
-            captureException(
+            this.captureException(
               new Error(
                 `Attempt to get permission specifications failed because their were ${accounts.length} accounts, but ${identitiesCount} identities, and the ${keyringTypesWithMissingIdentities} keyrings included accounts with missing identities. Meanwhile, there are ${accountTrackerCount} accounts in the account tracker.`,
               ),