From a81234635fc1727c3664a74aed64fe15eedc69c1 Mon Sep 17 00:00:00 2001
From: Baptiste Grob <60621355+baptiste-grob@users.noreply.github.com>
Date: Wed, 25 Nov 2020 13:16:11 +0100
Subject: [PATCH] fix: do not attempt to load bugsnag if no api key is set

---
 app/assets/javascripts/services/errorReporting.ts | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/app/assets/javascripts/services/errorReporting.ts b/app/assets/javascripts/services/errorReporting.ts
index 8577376c9..50c439db5 100644
--- a/app/assets/javascripts/services/errorReporting.ts
+++ b/app/assets/javascripts/services/errorReporting.ts
@@ -4,6 +4,11 @@ import { storage, StorageKey } from './localStorage';
 import Bugsnag from '@bugsnag/js';
 
 declare const __VERSION__: string;
+declare global {
+  interface Window {
+    _bugsnag_api_key?: string;
+  }
+}
 
 function redactFilePath(line: string): string {
   const fileName = line.match(/\w+\.(html|js)/)?.[0];
@@ -16,13 +21,16 @@ function redactFilePath(line: string): string {
 }
 
 export function startErrorReporting() {
-  if (storage.get(StorageKey.DisableErrorReporting)) {
+  if (
+    storage.get(StorageKey.DisableErrorReporting) ||
+    !window._bugsnag_api_key
+  ) {
     SNLog.onError = console.error;
     return;
   }
   try {
     Bugsnag.start({
-      apiKey: (window as any)._bugsnag_api_key,
+      apiKey: window._bugsnag_api_key,
       appType: isDesktopApplication() ? 'desktop' : 'web',
       appVersion: __VERSION__,
       collectUserIp: false,