From e3fda83ab209af7836ba93bfaba215c271d73e8a Mon Sep 17 00:00:00 2001
From: kumavis <aaron@kumavis.me>
Date: Sat, 20 Oct 2018 02:22:50 -0400
Subject: sentry - replace raven-js with sentry/browser

---
 app/scripts/lib/setupSentry.js | 91 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 91 insertions(+)
 create mode 100644 app/scripts/lib/setupSentry.js

(limited to 'app/scripts/lib/setupSentry.js')

diff --git a/app/scripts/lib/setupSentry.js b/app/scripts/lib/setupSentry.js
new file mode 100644
index 000000000..aa8d72194
--- /dev/null
+++ b/app/scripts/lib/setupSentry.js
@@ -0,0 +1,91 @@
+const Sentry = require('@sentry/browser')
+const METAMASK_DEBUG = process.env.METAMASK_DEBUG
+const extractEthjsErrorMessage = require('./extractEthjsErrorMessage')
+const SENTRY_DSN_PROD = 'https://3567c198f8a8412082d32655da2961d0@sentry.io/273505'
+const SENTRY_DSN_DEV = 'https://f59f3dd640d2429d9d0e2445a87ea8e1@sentry.io/273496'
+
+module.exports = setupSentry
+
+// Setup sentry remote error reporting
+function setupSentry (opts) {
+  const { release } = opts
+  let sentryTarget
+  // detect brave
+  const isBrave = Boolean(window.chrome.ipcRenderer)
+
+  if (METAMASK_DEBUG) {
+    console.log('Setting up Sentry Remote Error Reporting: SENTRY_DSN_DEV')
+    sentryTarget = SENTRY_DSN_DEV
+  } else {
+    console.log('Setting up Sentry Remote Error Reporting: SENTRY_DSN_PROD')
+    sentryTarget = SENTRY_DSN_PROD
+  }
+
+  Sentry.init({
+    dsn: sentryTarget,
+    debug: METAMASK_DEBUG,
+    release,
+    beforeSend: (report) => rewriteReport(report),
+  })
+
+  Sentry.configureScope(scope => {
+    scope.setExtra('isBrave', isBrave)
+  })
+
+  function rewriteReport(report) {
+    try {
+      // simplify certain complex error messages (e.g. Ethjs)
+      simplifyErrorMessages(report)
+      // modify report urls
+      rewriteReportUrls(report)
+    } catch (err) {
+      console.warn(err)
+    }
+  }
+
+  return Sentry
+}
+
+function simplifyErrorMessages (report) {
+  rewriteErrorMessages(report, (errorMessage) => {
+    // simplify ethjs error messages
+    errorMessage = extractEthjsErrorMessage(errorMessage)
+    // simplify 'Transaction Failed: known transaction'
+    if (errorMessage.indexOf('Transaction Failed: known transaction') === 0) {
+      // cut the hash from the error message
+      errorMessage = 'Transaction Failed: known transaction'
+    }
+    return errorMessage
+  })
+}
+
+function rewriteErrorMessages (report, rewriteFn) {
+  // rewrite top level message
+  if (typeof report.message === 'string') report.message = rewriteFn(report.message)
+  // rewrite each exception message
+  if (report.exception && report.exception.values) {
+    report.exception.values.forEach(item => {
+      if (typeof item.value === 'string') item.value = rewriteFn(item.value)
+    })
+  }
+}
+
+function rewriteReportUrls (report) {
+  // update request url
+  report.request.url = toMetamaskUrl(report.request.url)
+  // update exception stack trace
+  if (report.exception && report.exception.values) {
+    report.exception.values.forEach(item => {
+      item.stacktrace.frames.forEach(frame => {
+        frame.filename = toMetamaskUrl(frame.filename)
+      })
+    })
+  }
+}
+
+function toMetamaskUrl (origUrl) {
+  const filePath = origUrl.split(location.origin)[1]
+  if (!filePath) return origUrl
+  const metamaskUrl = `metamask${filePath}`
+  return metamaskUrl
+}
-- 
cgit 


From 73ec4e66cb7a476d01371a61692b0d8d9224da04 Mon Sep 17 00:00:00 2001
From: kumavis <aaron@kumavis.me>
Date: Sat, 20 Oct 2018 03:14:59 -0400
Subject: sentry - include app state in ui errors

---
 app/scripts/lib/setupSentry.js | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

(limited to 'app/scripts/lib/setupSentry.js')

diff --git a/app/scripts/lib/setupSentry.js b/app/scripts/lib/setupSentry.js
index aa8d72194..69042bc19 100644
--- a/app/scripts/lib/setupSentry.js
+++ b/app/scripts/lib/setupSentry.js
@@ -8,7 +8,7 @@ module.exports = setupSentry
 
 // Setup sentry remote error reporting
 function setupSentry (opts) {
-  const { release } = opts
+  const { release, getState } = opts
   let sentryTarget
   // detect brave
   const isBrave = Boolean(window.chrome.ipcRenderer)
@@ -38,9 +38,15 @@ function setupSentry (opts) {
       simplifyErrorMessages(report)
       // modify report urls
       rewriteReportUrls(report)
+      // append app state
+      if (getState) {
+        const appState = getState()
+        report.extra.appState = appState
+      }
     } catch (err) {
       console.warn(err)
     }
+    return report
   }
 
   return Sentry
-- 
cgit