fix: remove Bugsnag

It is causing crashes in production
2anki · Oct 31, 2023 · f6995b9 · f6995b9
1 parent bee3535
commit f6995b9
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 161 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -31,8 +31,6 @@
   "license": "MIT",
   "dependencies": {
     "@2anki/csv-to-apkg": "^1.4.4",
-    "@bugsnag/js": "^7.20.2",
-    "@bugsnag/plugin-express": "^7.19.0",
     "@notionhq/client": "^2.2.9",
     "@sendgrid/mail": "^7.7.0",
     "aws-sdk": "^2.1359.0",

diff --git a/src/lib/error/CrashReporter.ts b/src/lib/error/CrashReporter.ts
diff --git a/src/lib/error/getErrorReporterApiKey.ts b/src/lib/error/getErrorReporterApiKey.ts
diff --git a/src/lib/error/sendError.ts b/src/lib/error/sendError.ts
@@ -1,12 +1,7 @@
-/* eslint-disable import/no-extraneous-dependencies */
-import Bugsnag from '@bugsnag/js';
-
 export const sendError = (error: unknown) => {
   if (error instanceof Error) {
-    if (process.env.LOCAL_DEV === 'true') {
-      console.error(error);
-    } else if (process.env.NODE_ENV === 'production') {
-      Bugsnag.notify(error);
-    }
+    console.error(error);
+  } else {
+    console.log('unknown error '+ error);
   }
 };
diff --git a/src/server.ts b/src/server.ts
@@ -28,12 +28,10 @@ import favoriteRouter from './routes/FavoriteRouter';
 import templatesRouter from './routes/TemplatesRouter';
 import defaultRouter from './routes/DefaultRouter';
 
-import CrashReporter from './lib/error/CrashReporter';
 import { sendError } from './lib/error/sendError';
 
 import { isStaging } from './lib/isStaging';
 import { getDatabase, setupDatabase } from './data_layer';
-import { getErrorReporterApiKey } from './lib/error/getErrorReporterApiKey';
 function registerSignalHandlers(server: http.Server) {
   process.on('uncaughtException', sendError);
   process.on('SIGTERM', () => {
@@ -56,11 +54,6 @@ const serve = async () => {
   app.use(express.json({ limit: '1000mb' }) as RequestHandler);
   app.use(cookieParser());
 
-  const errorReporterApiKey = getErrorReporterApiKey();
-  if (errorReporterApiKey) {
-    CrashReporter.Configure(app, errorReporterApiKey);
-  }
-
   if (isStaging()) {
     app.use(morgan('combined') as RequestHandler);
   }
@@ -98,10 +91,6 @@ const serve = async () => {
     }
   );
 
-  if (errorReporterApiKey) {
-    CrashReporter.AddErrorHandler(app);
-  }
-
   app.use(
     (
       err: Error,