Remove cache on get report

bcgsc · Nov 15, 2023 · 68dc978 · 68dc978
1 parent 09efab2
commit 68dc978
Showing 1 changed file with 0 additions and 11 deletions.
diff --git a/app/middleware/report.js b/app/middleware/report.js
@@ -29,16 +29,8 @@ module.exports = async (req, res, next, ident) => {
     },
   ];
 
-  // Check cache for report
-  const key = `/reports/${ident}`;
   let result;
 
-  try {
-    result = await cache.get(key);
-  } catch (error) {
-    logger.error(`Error during report cache get ${error}`);
-  }
-
   if (result) {
     // Build Sequelize model from cached string without calling db
     result = db.models.report.build(JSON.parse(result), {
@@ -68,9 +60,6 @@ module.exports = async (req, res, next, ident) => {
       logger.error(`User does not have non-production access to ${ident}`);
       return res.status(HTTP_STATUS.FORBIDDEN).json({error: {message: 'User does not have access to Non-Production reports'}});
     }
-
-    // Add result to cache
-    cache.set(key, JSON.stringify(result), 'EX', 14400);
   }
 
   // Add report to request