chore: Remove Rollbar usage (#204)

* chore: Remove Rollbar usage * feat: Log previously reported errors
decentraland · Aug 29, 2023 · c28860e · c28860e
1 parent bb39b1a
commit c28860e
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 217 deletions.
diff --git a/electron/main.ts b/electron/main.ts
@@ -7,7 +7,6 @@ import { getAppTitle, getAppBasePath } from './helpers'
 import { createWindow, hideWindowInTray, loadDecentralandWeb, onOpenUrl, showWindowAndHideTray } from './window'
 import { LauncherConfig, LauncherPaths } from './types'
 import { isTrustedCertificate } from './certificateChecker'
-import { reportError, initializeRollbar } from './rollbar'
 import fs = require('fs')
 
 const defaultConfig: LauncherConfig = {
@@ -34,7 +33,6 @@ class MainApp {
 }
 
 // all uncaught exceptions are being sent automatically
-initializeRollbar()
 
 initializeCrashReport()
 
@@ -68,9 +66,8 @@ console.log('Config:', main.config)
 console.log('OS:', osName)
 
 if (getOSName() === null) {
-  reportError('OS not supported', () => {
-    exit(1)
-  })
+  console.error('OS not supported')
+  exit(1)
 }
 
 const launcherPaths: LauncherPaths = {
@@ -215,7 +212,7 @@ app
     })
   })
   .catch(async (error) => {
-    reportError(error)
+    console.error(`Error starting app: ${error}`)
   })
 
 function initializeCrashReport() {

diff --git a/electron/rollbar.ts b/electron/rollbar.ts
diff --git a/electron/updater.ts b/electron/updater.ts
@@ -136,12 +136,9 @@ const reportFatalError = async (sender: WebContents, message: string) => {
 const reportCrash = async (sender: WebContents, message: string) => {
   const path = JSON.stringify({ playerlogpath: getPlayerLogPath() })
   const data = JSON.stringify(`Player log:\n${getPlayerLog()}`)
-  const code = `window.Rollbar.error(${data}, ${path})`
   console.log(`reportCrash path: ${path}`)
 
   try {
-    await sender.executeJavaScript(code)
-
     await reportFatalError(sender, `Renderer unexpected exit: ${message}`)
   } catch (e) {
     console.error(`Report crash, error: ${e}`)
@@ -166,7 +163,8 @@ const registerExecuteProcessEvent = (rendererPath: string, executablePath: strin
 
       // We didn't find a way to get this windows store app package path dynamically
       if (process.windowsStore) {
-        rendererPath = process.env.LOCALAPPDATA +
+        rendererPath =
+          process.env.LOCALAPPDATA +
           `\\Packages\\DecentralandFoundation.Decentraland_4zmdhd0rz3xz8\\LocalCache\\Roaming\\explorer-desktop-launcher\\renderer\\`
       }