From 0553bfe2ffaa782677fbf7ceb046571b1f2a6be7 Mon Sep 17 00:00:00 2001
From: Stuart Taylor <staylor1012@gmail.com>
Date: Thu, 17 May 2018 15:26:44 +0100
Subject: [PATCH] refactor(server): Remove opbeat from dev tools

---
 package.json                         |  1 -
 server/middlewares/error-reporter.js | 20 ++------------------
 server/server.js                     |  9 ---------
 3 files changed, 2 insertions(+), 28 deletions(-)

diff --git a/package.json b/package.json
index e3099a89b8..526a9ab311 100644
--- a/package.json
+++ b/package.json
@@ -99,7 +99,6 @@
     "nodemailer-ses-transport": "^1.3.0",
     "normalize-url": "^1.3.1",
     "normalizr": "2.2.1",
-    "opbeat": "^4.14.0",
     "passport": "^0.4.0",
     "passport-facebook": "^2.0.0",
     "passport-github": "^1.0.0",
diff --git a/server/middlewares/error-reporter.js b/server/middlewares/error-reporter.js
index 5b3aaba86b..2a7be6c379 100644
--- a/server/middlewares/error-reporter.js
+++ b/server/middlewares/error-reporter.js
@@ -1,4 +1,3 @@
-import opbeat from 'opbeat';
 import debug from 'debug';
 
 import {
@@ -8,7 +7,6 @@ import {
 
 const log = debug('fcc:middlewares:error-reporter');
 
-const isOpbeatDisabled = !opbeat.appId;
 export default function errrorReporter() {
   if (process.env.NODE_ENV !== 'production') {
     return (err, req, res, next) => {
@@ -18,21 +16,7 @@ export default function errrorReporter() {
         log(handled.message);
       }
       next(err);
-     };
+    };
   }
-  return (err, req, res, next) => {
-    // handled errors do not need to be reported
-    // the report a message and redirect the user
-    if (
-      isOpbeatDisabled ||
-      isHandledError(err) ||
-      // errors with status codes shouldn't be reported
-      // as they are usually user messages
-      err.statusCode ||
-      err.status
-    ) {
-      return next(err);
-    }
-    return opbeat.captureError(err, { request: req }, () => next(err));
-  };
+  return (err, req, res, next) => next(err);
 }
diff --git a/server/server.js b/server/server.js
index 216e680a3c..7b31de9e8c 100755
--- a/server/server.js
+++ b/server/server.js
@@ -1,15 +1,6 @@
 require('dotenv').load();
 require('./utils/webpack-code-split-polyfill');
 
-if (process.env.OPBEAT_ID) {
-  console.log('loading opbeat');
-  require('opbeat').start({
-    appId: process.env.OPBEAT_ID,
-    organizationId: process.env.OPBEAT_ORG_ID,
-    secretToken: process.env.OPBEAT_SECRET
-  });
-}
-
 const _ = require('lodash');
 const Rx = require('rx');
 const loopback = require('loopback');