feat: update logging

Author: eisbilir

app-routes.js

@@ -108,7 +108,10 @@ module.exports = (app) => {
           }
         });
       }
-
+      actions.push((req, res, next) => {
+        logger.info(`Done request handling, ${req.signature}`);
+        next();
+      });
       actions.push(method);
       app[verb](`/${config.API_VERSION}${path}`, helper.autoWrapExpress(actions));
     });

src/common/logger.js

@@ -30,10 +30,16 @@ logger.logFullError = (err, signature) => {
   if (signature) {
     logger.error(`Error happened in ${signature}`);
   }
-  logger.error(util.inspect(err));
-  if (!err.logged) {
-    logger.error(err.stack);
-    err.logged = true;
+  if (err.isJoi) {
+    logger.error(
+      `${e.name} details: ${JSON.stringify(e.details)} input:${JSON.stringify(e._object)}`
+    );
+  } else if (err.isAxiosError) {
+    logger.error(`${err.message} - ${err.response.data}`);
+  } else if (err.httpStatus) {
+    logger.error(err.message);
+  } else {
+    logger.error(util.inspect(err));
   }
 };
 

src/services/ChallengeService.js

@@ -1420,7 +1420,7 @@ async function updateChallenge(currentUser, challengeId, data) {
 
   // Remove fields from data that are not allowed to be updated and that match the existing challenge
   data = sanitizeData(sanitizeChallenge(data), challenge);
-  console.debug("Sanitized Data:", data);
+  logger.debug("Sanitized Data:", JSON.stringify(data));
 
   await validateChallengeUpdateRequest(currentUser, challenge, data);