Merge pull request #587 from topcoder-platform/debug/logs

fix: do not log es queries, makes logs very busy

Author: rakibansary

src/common/helper.js

@@ -197,7 +197,6 @@ async function getById(modelName, id) {
       .eq(id)
       .exec((err, result) => {
         if (err) {
-          console.log("rejecting error", err);
           return reject(err);
         }
         if (result.length > 0) {

src/services/ChallengeService.js

@@ -158,7 +158,6 @@ async function searchByLegacyId(currentUser, legacyId, page, perPage) {
       },
     },
   };
-  logger.debug(`es Query ${JSON.stringify(esQuery)}`);
   let docs;
   try {
     docs =
@@ -847,8 +846,6 @@ async function searchChallenges(currentUser, criteria) {
       ],
     },
   };
-
-  logger.debug(`es Query ${JSON.stringify(esQuery, null, 4)}`);
   // Search with constructed query
   let docs;
   try {
@@ -858,7 +855,6 @@ async function searchChallenges(currentUser, criteria) {
         : (await esClient.search(esQuery)).body;
   } catch (e) {
     // Catch error when the ES is fresh and has no data
-    logger.error(`Query Error from ES ${JSON.stringify(e, null, 4)}`);
     docs = {
       hits: {
         total: 0,
@@ -1758,7 +1754,6 @@ async function updateChallenge(currentUser, challengeId, data) {
   }
 
   if (data.winners && data.winners.length && data.winners.length > 0) {
-    console.log("Request to validate winners", data.winners, challengeId);
     await validateWinners(data.winners, challengeId);
   }
 

src/services/PhaseService.js

@@ -102,7 +102,6 @@ async function update(phaseId, data, isFull) {
     const { items } = await phaseDomain.scan({
       criteria: getScanCriteria({ name: data.name }),
     });
-    console.log("items", items);
     if (items.length > 0)
       throw new errors.ConflictError(`Phase with name ${phase.name} already exists`);
   }