Fix sentry errors (#256)

src/accounts/ui.tsx

@@ -23,7 +23,10 @@ import { LedgerSignRequest } from 'ledger/types';
 import { ledgerService } from 'ledger/service';
 import { initUiSentry } from 'sentry';
 
-initUiSentry('accounts');
+initUiSentry({
+  ignoreErrorContext: 'beforeSendAccounts',
+  source: 'accounts',
+});
 
 log.setDefaultLevel(KEEPERWALLET_DEBUG ? 'debug' : 'warn');
 

src/background.js

@@ -77,6 +77,10 @@ Sentry.init({
     const backgroundService = await bgPromise;
 
     const shouldIgnore =
+      backgroundService.remoteConfigController.shouldIgnoreError(
+        'beforeSend',
+        message
+      ) ||
       backgroundService.remoteConfigController.shouldIgnoreError(
         'beforeSendBackground',
         message
@@ -1117,7 +1121,9 @@ class BackgroundService extends EventEmitter {
       account,
       network: this._getCurrentNetwork(state.selectedAccount),
       messages,
-      txVersion: getTxVersions(state.selectedAccount.type),
+      txVersion: getTxVersions(
+        state.selectedAccount ? state.selectedAccount.type : 'seed'
+      ),
     };
   }
 

src/constants.ts

@@ -125,8 +125,16 @@ export const IGNORE_ERRORS_CONFIG_UPDATE_INTERVAL = 60000;
 
 export const DEFAULT_IGNORE_ERRORS_CONFIG = {
   ignoreAll: false,
-  beforeSendBackground: ['Failed to fetch'],
-  beforeSendPopup: ['Failed to fetch'],
+  beforeSend: [
+    'An operation that changes interface state is in progress',
+    'Failed to fetch',
+    'NetworkError when attempting to fetch resource',
+    'No device selected',
+    'The operation was aborted',
+  ],
+  beforeSendAccounts: [],
+  beforeSendBackground: [],
+  beforeSendPopup: [],
   contentScriptApprove: [] as string[],
   popupApprove: [] as string[],
 };

src/controllers/AssetInfoController.js

@@ -347,10 +347,6 @@ export class AssetInfoController {
       if (resp.ok) {
         this.suspiciousAssets = (await resp.text()).split('\n').sort();
         this.suspiciousLastUpdated = new Date().getTime();
-      } else {
-        if (resp.status < 500 && resp.status !== 403) {
-          throw new Error(await resp.text());
-        }
       }
     }
 

src/sentry/index.ts

@@ -2,7 +2,13 @@ import { KEEPERWALLET_DEBUG } from '../constants';
 import * as Sentry from '@sentry/react';
 import backgroundService from 'ui/services/Background';
 
-export function initUiSentry(source: 'popup' | 'accounts') {
+export function initUiSentry({
+  ignoreErrorContext,
+  source,
+}: {
+  ignoreErrorContext: 'beforeSendAccounts' | 'beforeSendPopup';
+  source: 'popup' | 'accounts';
+}) {
   return Sentry.init({
     dsn: __SENTRY_DSN__,
     environment: __SENTRY_ENVIRONMENT__,
@@ -25,10 +31,12 @@ export function initUiSentry(source: 'popup' | 'accounts') {
           ? hint.originalException.message
           : String(hint.originalException);
 
-      const shouldIgnore = await backgroundService.shouldIgnoreError(
-        'beforeSendPopup',
-        message
-      );
+      const [shouldIgnoreGlobal, shouldIgnoreContext] = await Promise.all([
+        backgroundService.shouldIgnoreError('beforeSend', message),
+        backgroundService.shouldIgnoreError(ignoreErrorContext, message),
+      ]);
+
+      const shouldIgnore = shouldIgnoreGlobal || shouldIgnoreContext;
 
       if (shouldIgnore) {
         return null;

src/ui.tsx

@@ -25,7 +25,10 @@ import { initUiSentry } from 'sentry';
 
 const isNotificationWindow = window.location.pathname === '/notification.html';
 
-initUiSentry('popup');
+initUiSentry({
+  ignoreErrorContext: 'beforeSendPopup',
+  source: 'popup',
+});
 
 log.setDefaultLevel(KEEPERWALLET_DEBUG ? 'debug' : 'warn');
 

src/ui/components/pages/BackupSeed.tsx

@@ -3,10 +3,9 @@ import * as React from 'react';
 import { Trans } from 'react-i18next';
 import { Button, Copy, Modal } from '../ui';
 import { PAGES } from '../../pageConfig';
-import { useAccountsSelector, useAppDispatch } from 'accounts/store';
+import { useAccountsSelector } from 'accounts/store';
 
 export function BackUpSeed({ setTab }) {
-  const dispatch = useAppDispatch();
   const [showCopy, setShowCopy] = React.useState<boolean>(false);
   const newAccount = useAccountsSelector(state => state.localState.newAccount);
 
@@ -47,12 +46,12 @@ export function BackUpSeed({ setTab }) {
         id="continue"
         className="margin-main-big"
         type="submit"
-        onClick={() => dispatch(setTab(PAGES.CONFIRM_BACKUP))}
+        onClick={() => setTab(PAGES.CONFIRM_BACKUP)}
       >
         <Trans i18nKey="backupSeed.continue">Continue</Trans>
       </Button>
 
-      <Button id="cancelCreation" onClick={() => dispatch(setTab(PAGES.ROOT))}>
+      <Button id="cancelCreation" onClick={() => setTab(PAGES.ROOT)}>
         <Trans i18nKey="backupSeed.cancel">Cancel creation</Trans>
       </Button>
 

src/ui/components/pages/swap/swap.tsx

@@ -227,6 +227,12 @@ export function Swap({ setTab }: Props) {
                         }
                         return;
                       }
+
+                      if (/Request is rejected on ledger/i.test(errMessage)) {
+                        setSwapErrorMessage(errMessage);
+                        setIsSwapInProgress(false);
+                        return;
+                      }
                     }
 
                     setSwapErrorMessage(errMessage || t('swap.failMessage'));