Jordan+Fabo/1626 update sentry

CHANGELOG.md

@@ -78,6 +78,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 - [\#1638](https://github.com/cosmos/voyager/issues/1638) removed account password from the state and now user has to input it on every transaction @fedekunze
 - [\#1655](https://github.com/cosmos/voyager/issues/1655) Text and Textarea fields trimmed @sabau
 - [\#1686](https://github.com/cosmos/voyager/issues/1686) Changed proposals from array to object @sabau
+- [\#1626](https://github.com/cosmos/voyager/issues/1626) upgraded from raven to sentry/browser @jbibla
 
 ### Fixed
 

app/src/main/index.js

@@ -7,7 +7,7 @@ const https = require(`https`)
 let { join, relative } = require(`path`)
 let childProcess = require(`child_process`)
 let semver = require(`semver`)
-let Raven = require(`raven`)
+const Sentry = require("@sentry/node")
 const readline = require(`readline`)
 let axios = require(`axios`)
 
@@ -222,7 +222,7 @@ function startProcess(name, args, env) {
       console.error(...errorMessage) // also output to console for easier debugging
       handleCrash(error)
 
-      Raven.captureException(error)
+      Sentry.captureException(error)
     }
   })
 
@@ -382,12 +382,12 @@ if (!TEST) {
   // on uncaught exceptions we wait so the sentry event can be sent
   process.on(`uncaughtException`, async function(error) {
     logError(`[Uncaught Exception]`, error)
-    Raven.captureException(error)
+    Sentry.captureException(error)
     handleCrash(error)
   })
   process.on(`unhandledRejection`, async function(error) {
     logError(`[Unhandled Promise Rejection]`, error)
-    Raven.captureException(error)
+    Sentry.captureException(error)
     handleCrash(error)
   })
 }
@@ -399,11 +399,13 @@ const eventHandlers = {
   },
 
   "error-collection": (event, optin) => {
-    Raven.uninstall()
-      .config(optin ? config.sentry_dsn : ``, {
-        captureUnhandledRejections: false
+    if (optin) {
+      Sentry.init({
+        dsn: config.sentry_dsn,
+        release: `[email protected]`
       })
-      .install()
+      Sentry.captureException(new Error("Good bye"))
+    }
   },
 
   mocked: value => {
@@ -595,8 +597,12 @@ const checkGaiaCompatibility = async gaiacliVersionPath => {
 }
 
 async function main() {
-  // we only enable error collection after users opted in
-  Raven.config(``, { captureUnhandledRejections: false }).install()
+  // Sentry is used for automatic error reporting. It is turned off by default.
+  Sentry.init({
+    dsn: config.sentry_dsn,
+    release: `[email protected]`
+  })
+  Sentry.captureException(new Error("TESTING SENTRY"))
 
   let appVersionPath = join(root, `app_version`)
   let genesisPath = join(root, `genesis.json`)

app/src/renderer/main.js

@@ -5,7 +5,7 @@ import Electron from "vue-electron"
 import Router from "vue-router"
 import Tooltip from "vue-directive-tooltip"
 import Vuelidate from "vuelidate"
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 import { ipcRenderer, remote } from "electron"
 
 import App from "./App"
@@ -21,23 +21,28 @@ let store
 let node
 let router
 
-// Raven serves automatic error reporting. It is turned off by default
-Raven.config(``).install()
+// Sentry is used for automatic error reporting. It is turned off by default.
+Sentry.init({})
+
+// this will pass the state to Sentry when errors are sent.
+Sentry.configureScope(scope => {
+  scope.setExtra(Store.state)
+})
 
 // handle uncaught errors
 window.addEventListener(`unhandledrejection`, function(event) {
-  Raven.captureException(event.reason)
+  Sentry.captureException(event.reason)
 })
 window.addEventListener(`error`, function(event) {
-  Raven.captureException(event.reason)
+  Sentry.captureException(event.reason)
 })
 
 Vue.config.errorHandler = (error, vm, info) => {
   console.error(`An error has occurred: ${error}
 
 Guru Meditation #${info}`)
 
-  Raven.captureException(error)
+  Sentry.captureException(error)
 
   if (store.state.devMode) {
     throw error

app/src/renderer/vuex/modules/blockchain.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 
 export default ({ node }) => {
   const state = {
@@ -67,7 +67,7 @@ export default ({ node }) => {
           title: `Error fetching block information`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.loading = false
         state.error = error
         return null

app/src/renderer/vuex/modules/connection.js

@@ -1,5 +1,5 @@
 import { ipcRenderer, remote } from "electron"
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 import { sleep } from "scripts/common.js"
 
 const config = remote.getGlobal(`config`)
@@ -86,7 +86,7 @@ export default function({ node }) {
         { query: `tm.event = 'NewBlockHeader'` },
         (error, event) => {
           if (error) {
-            Raven.captureException(error)
+            Sentry.captureException(error)
             return console.error(`error subscribing to headers`, error)
           }
           dispatch(`setLastHeader`, event.data.value.header)
@@ -126,7 +126,7 @@ export default function({ node }) {
       }, timeout)
       node.rpc.status(error => {
         if (error) {
-          Raven.captureException(error)
+          Sentry.captureException(error)
           console.error(`Couldn't get status via RPC:`, error)
           return
         }

app/src/renderer/vuex/modules/delegates.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 import BN from "bignumber.js"
 import { ratToBigNumber } from "scripts/common"
 import num from "scripts/num"
@@ -106,7 +106,7 @@ export default ({ node }) => {
           title: `Error fetching validators`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         commit(`setDelegateLoading`, false)
         state.error = error
         return []

app/src/renderer/vuex/modules/delegation.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 import { calculateShares } from "scripts/common"
 
 export default ({ node }) => {
@@ -149,7 +149,7 @@ export default ({ node }) => {
           title: `Error fetching delegations`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
 

app/src/renderer/vuex/modules/governance/deposits.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 
 export default ({ node }) => {
   const state = {
@@ -30,7 +30,7 @@ export default ({ node }) => {
           title: `Error fetching deposits on proposals`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
     },

app/src/renderer/vuex/modules/governance/parameters.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 
 export default ({ node }) => {
   const emptyState = {
@@ -38,7 +38,7 @@ export default ({ node }) => {
           title: `Error fetching governance parameters`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
       state.loading = false

app/src/renderer/vuex/modules/governance/proposals.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 import Vue from "vue"
 
 export default ({ node }) => {
@@ -55,7 +55,7 @@ export default ({ node }) => {
           title: `Error fetching proposals`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
     },
@@ -72,7 +72,7 @@ export default ({ node }) => {
           title: `Error querying proposal with id #${proposal_id}`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
     },

app/src/renderer/vuex/modules/governance/votes.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 
 export default ({ node }) => {
   const state = {
@@ -30,7 +30,7 @@ export default ({ node }) => {
           title: `Error fetching votes`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
     },

app/src/renderer/vuex/modules/keybase.js

@@ -1,5 +1,5 @@
 import axios from "axios"
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 
 export default ({}) => {
   const emptyState = {
@@ -60,7 +60,7 @@ export default ({}) => {
           title: `Error fetching keybase information for validators`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
     }

app/src/renderer/vuex/modules/parameters.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 
 export default ({ node }) => {
   const emptyState = {
@@ -33,7 +33,7 @@ export default ({ node }) => {
           title: `Error fetching staking parameters`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
     }

app/src/renderer/vuex/modules/pool.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 
 export default ({ node }) => {
   const emptyState = {
@@ -37,7 +37,7 @@ export default ({ node }) => {
           title: `Error fetching staking pool information`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
     }

app/src/renderer/vuex/modules/transactions.js

@@ -1,6 +1,6 @@
 import fp from "lodash/fp"
 import { uniqBy } from "lodash"
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 export default ({ node }) => {
   let emptyState = {
     loading: false,
@@ -76,7 +76,7 @@ export default ({ node }) => {
           title: `Error getting transactions`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
     },

app/src/renderer/vuex/modules/user.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 import { ipcRenderer, remote } from "electron"
 import enableGoogleAnalytics from "../../google-analytics.js"
 const config = remote.getGlobal(`config`)
@@ -65,7 +65,7 @@ export default ({ node }) => {
         let keys = await node.keys.values()
         commit(`setAccounts`, keys)
       } catch (error) {
-        Raven.captureException(error)
+        Sentry.captureException(error)
         commit(`notifyError`, {
           title: `Couldn't read keys`,
           body: error.message
@@ -147,12 +147,10 @@ export default ({ node }) => {
         state.errorCollection
       )
 
-      Raven.uninstall()
-        .config(state.errorCollection ? config.sentry_dsn_public : ``)
-        .install()
       if (state.errorCollection) {
-        console.log(`Analytics enabled in browser`)
+        Sentry.init({ dsn: config.sentry_dsn })
         enableGoogleAnalytics(config.google_analytics_uid)
+        console.log(`Analytics and error reporting have been enabled`)
         window.analytics &&
           window.analytics.send(`pageview`, {
             dl: window.location.pathname

app/src/renderer/vuex/modules/validators.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 
 export default ({ node }) => {
   const emptyState = {
@@ -45,7 +45,7 @@ export default ({ node }) => {
           title: `Error fetching validator set`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
     },

app/src/renderer/vuex/modules/wallet.js

@@ -1,4 +1,4 @@
-import Raven from "raven-js"
+import * as Sentry from "@sentry/browser"
 import fs from "fs-extra"
 import { join } from "path"
 import { remote } from "electron"
@@ -83,7 +83,7 @@ export default ({ node }) => {
           title: `Error fetching balances`,
           body: error.message
         })
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
       }
     },
@@ -107,7 +107,7 @@ export default ({ node }) => {
       }
       if (maxIterations === 0) {
         const error = new Error(`Couldn't load genesis at path ${genesisPath}`)
-        Raven.captureException(error)
+        Sentry.captureException(error)
         state.error = error
         return
       }
@@ -145,7 +145,7 @@ export default ({ node }) => {
 
       function onTx(error, event) {
         if (error) {
-          Raven.captureException(error)
+          Sentry.captureException(error)
           console.error(`error subscribing to transactions`, error)
           return
         }