Don't require Bugsnag

Fixes #14.
piratepx · Dec 15, 2020 · 3dcd2e4 · 3dcd2e4
1 parent 8441449
commit 3dcd2e4
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 7 deletions.
diff --git a/api/plugins/authenticate_project.js b/api/plugins/authenticate_project.js
@@ -28,7 +28,9 @@ module.exports = fp(async (fastify, { allowShared = false }) => {
         return false
       }
 
-      bugsnag.setUser(project.id, project.user.email, project.name)
+      if (bugsnag) {
+        bugsnag.setUser(project.id, project.user.email, project.name)
+      }
 
       project.is_shared = isShared
 

diff --git a/api/plugins/error_handler/index.js b/api/plugins/error_handler/index.js
@@ -9,7 +9,10 @@ module.exports = async (error, request, reply) => {
   }
 
   if (reply.statusCode >= 500) {
-    bugsnag.notify(error)
+    if (bugsnag) {
+      bugsnag.notify(error)
+    }
+
     reply.log.error({ req: request, res: reply, err: error }, error.message)
   } else if (reply.statusCode >= 400) {
     reply.log.info({ res: reply, err: error }, error.message)

diff --git a/api/services/bugsnag.js b/api/services/bugsnag.js
@@ -2,8 +2,10 @@ const Bugsnag = require('@bugsnag/js')
 
 const package = require('@/../package')
 
-module.exports = Bugsnag.start({
-  apiKey: process.env.BUGSNAG_API_KEY,
-  appVersion: package.version,
-  enabledReleaseStages: ['production'],
-})
+module.exports = process.env.BUGSNAG_API_KEY
+  ? Bugsnag.start({
+      apiKey: process.env.BUGSNAG_API_KEY,
+      appVersion: package.version,
+      enabledReleaseStages: ['production'],
+    })
+  : null