From 89ddd4bb7e3f8614c04d5b43c91986f98f7d5892 Mon Sep 17 00:00:00 2001
From: mrugesh <1884376+raisedadead@users.noreply.github.com>
Date: Wed, 1 Jan 2020 10:53:11 +0530
Subject: [PATCH] fix(api): error reporter should have explicit env check
 (#38000)

---
 api-server/server/middlewares/error-reporter.js | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/api-server/server/middlewares/error-reporter.js b/api-server/server/middlewares/error-reporter.js
index 81a368b202..f3d338d421 100644
--- a/api-server/server/middlewares/error-reporter.js
+++ b/api-server/server/middlewares/error-reporter.js
@@ -27,7 +27,8 @@ ${JSON.stringify(error, null, 2)}
 };
 
 export function reportError(err) {
-  return process.env.FREECODECAMP_NODE_ENV === 'production'
+  return process.env.FREECODECAMP_NODE_ENV === 'production' &&
+    process.env.ERROR_REPORTER === 'true'
     ? reporter.error(err.message, err)
     : console.error(err);
 }
@@ -35,7 +36,7 @@ export function reportError(err) {
 export default function errrorReporter() {
   if (
     process.env.FREECODECAMP_NODE_ENV !== 'production' &&
-    process.env.ERROR_REPORTER
+    process.env.ERROR_REPORTER === 'true'
   ) {
     return (err, req, res, next) => {
       console.error(errTemplate(err, req));