From 7535a612c949408dae97fb17259a9635617b1b21 Mon Sep 17 00:00:00 2001
From: Pol Alvarez <pol.avms@gmail.com>
Date: Fri, 22 Apr 2022 17:05:29 +0200
Subject: [PATCH] removed comments

---
 src/api/route-services/pipeline-response.js | 9 ---------
 src/utils/authMiddlewares.js                | 1 -
 src/utils/hooks/send-notification.js        | 3 ---
 3 files changed, 13 deletions(-)

diff --git a/src/api/route-services/pipeline-response.js b/src/api/route-services/pipeline-response.js
index 5179aad8e..84a1069ab 100644
--- a/src/api/route-services/pipeline-response.js
+++ b/src/api/route-services/pipeline-response.js
@@ -118,9 +118,7 @@ class PipelineService {
   }
 
   static async qcResponse(io, message) {
-    console.log('starting qc response', message);
     AWSXRay.getSegment().addMetadata('message', message);
-    console.log('validating request');
     await validateRequest(message, 'PipelineResponse.v1.yaml');
 
     await pipelineHook.run(message);
@@ -130,20 +128,13 @@ class PipelineService {
 
     let output = null;
     // if there aren't errors proceed with the updates
-    console.log('and the rest is silence');
     if (!error && 'output' in message) {
       const { input: { sampleUuid, taskName } } = message;
 
-      console.log('getS3Output');
       output = await this.getS3Output(message);
-      console.log('updatePlotData');
       await this.updatePlotData(taskName, experimentId, output);
-      console.log('updateProcessingConfig');
       await this.updateProcessingConfig(taskName, experimentId, output, sampleUuid);
     }
-
-    console.log('and the rest is not silence');
-
     // we want to send the update to the subscribed both in successful and error case
     await this.sendUpdateToSubscribed(experimentId, message, output, error, io);
   }
diff --git a/src/utils/authMiddlewares.js b/src/utils/authMiddlewares.js
index ea16e438e..a3956d7a7 100644
--- a/src/utils/authMiddlewares.js
+++ b/src/utils/authMiddlewares.js
@@ -125,7 +125,6 @@ const checkAuthExpiredMiddleware = (req, res, next) => {
     return next(new UnauthenticatedError('token has expired'));
   }
 
-  console.log(`lcs checking ignore [${req.url}] [${req.method.toLowerCase()}]`);
   // check if we should ignore expired jwt token for this path and request type
   const longTimeoutEndpoints = [{ urlMatcher: /^\/v1\/experiments\/.{32}\/cellSets$/, method: 'PATCH' }];
   const isEndpointIgnored = longTimeoutEndpoints.some(
diff --git a/src/utils/hooks/send-notification.js b/src/utils/hooks/send-notification.js
index ec7012c25..b99bffd13 100644
--- a/src/utils/hooks/send-notification.js
+++ b/src/utils/hooks/send-notification.js
@@ -1,4 +1,3 @@
-// const jwt = require('jsonwebtoken');
 const getLogger = require('../getLogger');
 const { authenticationMiddlewareSocketIO } = require('../authMiddlewares');
 const getPipelineStatus = require('../../api/general-services/pipeline-status');
@@ -18,9 +17,7 @@ const sendNotification = async (message) => {
     return;
   }
 
-  logger.log('lcs verifying token ignoring expiration');
   const user = await authenticationMiddlewareSocketIO(authJWT, true);
-  logger.log('lcs user', user);
 
   const { experimentId } = message;
   const statusRes = await getPipelineStatus(experimentId, process);