fix: storage cache cross-process invalidation logic

Author: yash-pouranik

apps/dashboard-api/src/controllers/project.controller.js

@@ -780,6 +780,13 @@ module.exports.updateExternalConfig = async (req, res) => {
     if (!project)
       return res.status(404).json({ success: false, data: {}, message: "Project not found or access denied." });
 
+    await deleteProjectById(project._id.toString());
+    await deleteProjectByApiKeyCache(project.publishableKey);
+    await deleteProjectByApiKeyCache(project.secretKey);
+    if (updateData["resources.storage.config"]) {
+      storageRegistry.delete(project._id.toString());
+    }
+
     res.status(200).json({ success: true, data: {}, message: "External configuration updated successfully." });
   } catch (err) {
     if (err instanceof z.ZodError) {

packages/common/src/utils/storage.manager.js

@@ -115,11 +115,15 @@ async function getStorage(project) {
 
     const key = project._id.toString();
 
+    const currentConfigStr = project.resources?.storage?.config;
+
     // CACHE - REUSE EXISTING CLIENT
     if (storageRegistry.has(key)) {
         const entry = storageRegistry.get(key);
-        entry.lastUsed = Date.now();
-        return entry.client;
+        if (entry.configStr === currentConfigStr) {
+            entry.lastUsed = Date.now();
+            return entry.client;
+        }
     }
 
     let client;
@@ -160,6 +164,7 @@ async function getStorage(project) {
     // REGISTRY - REGISTER CLIENT FOR POOLING
     storageRegistry.set(key, {
         client,
+        configStr: currentConfigStr,
         lastUsed: Date.now(),
         isExternal: !!project.resources?.storage?.isExternal
     });