updates based on requested changed

functions/src/express_configs/express_open.ts

@@ -37,13 +37,8 @@ app.use(bodyParser.urlencoded({ extended: true }));
 // The error handler must be before any other error middleware and after all controllers
 app.use(
   Sentry.Handlers.errorHandler({
-    shouldHandleError(error) {
-      // Capture all errors over 400
-      if ((error.status as number) >= 400) {
-        return true;
-      }
-      return false;
-    },
+    // report the error to sentry if >=400
+    shouldHandleError: (error) => (error.status as number) >= 400,
   })
 );
 

functions/src/express_configs/express_secure.ts

@@ -62,7 +62,7 @@ function errorHandler(error: Error, request: Request, response: Response, next:
 app.use(errorHandler);
 
 // Automatically send uncaught exception errors to Sentry
-process.on("uncaughtException", (error) => Sentry.captureException(error));
+process.on("uncaughtException", (err) => Sentry.captureException(err));
 
 const checkJwt = jwt({
   secret: jwksRsa.expressJwtSecret({

functions/src/index.ts

@@ -16,7 +16,7 @@ import * as eventFunctions from "./events/events";
 import * as vanityFunctions from "./custom/vanity";
 import * as hacktoberfestFunctions from "./custom/hacktoberfest";
 import * as typeformFunctions from "./application/typeform";
-import * as errorFunctions from "./services/ErrorService";
+import * as errorFunctions from "./services/errorService";
 
 //this will match every call made to this api.
 app_secure.all("/", (request, response, next) => {