fixing the reused cached image when changing from satellite

NTNU-SmallSatLab · Aug 6, 2025 · 5d71dcc · 5d71dcc
1 parent da710c2
commit 5d71dcc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/backend/src/api/slack/controllers/slack.js b/backend/src/api/slack/controllers/slack.js
@@ -10,21 +10,23 @@ const fetch = require("node-fetch");
 
 let cachedImage = null;
 let cacheTimestamp = null;
+let cacheSat = null;
 
 module.exports = {
   fetchImages: async (ctx) => {
     const CACHE_DURATION = 60 * 1000; // 1 minute
     const now = Date.now();
+    const { satName } = ctx.request.body;
     if (
       cachedImage &&
       cacheTimestamp &&
-      now - cacheTimestamp < CACHE_DURATION
+      now - cacheTimestamp < CACHE_DURATION &&
+      cacheSat === satName
     ) {
       ctx.body = cachedImage;
       return;
     }
     try {
-      const { satName } = ctx.request.body;
       const message = await fetchImagesFromSlack.fetchImagesFromSlack(satName);
       const image = message ? message.files[0] : null;
       if (!image.public_url_shared) {

diff --git a/backend/src/api/slack/services/slack.js b/backend/src/api/slack/services/slack.js
@@ -25,8 +25,9 @@ module.exports = {
           console.log("No messages found in the channel.");
           return null;
         }
+        console.log(satellite);
         for (const message of result.messages) {
-          if (message.text && message.text.includes(satellite)) {
+          if (message.text.includes(satellite)) {
             if (!image) {
               image = message;
               break;