DINC0367584 :Issue with repository Id caching in multitenacy SDM CAP-JS

Now its cached based on subdomain and repositoryId combination
cap-js · Jan 13, 2025 · 0b192e9 · 0b192e9
1 parent 5182681
commit 0b192e9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/lib/sdm.js b/lib/sdm.js
@@ -39,8 +39,9 @@ module.exports = class SDMAttachmentsService extends (
 
   async checkRepositoryType(req) {
     const { repositoryId } = getConfigurations();
+    let subdomain = cds.context.user?.tokenInfo?.getPayload()?.ext_attr?.zdn;
     //check if repository is versionable
-    let repotype = cache.get(repositoryId) 
+    let repotype = cache.get(repositoryId+"_"+subdomain);
     let isVersioned;
     if (repotype == undefined) {
       const token = await getClientCredentialsToken(this.creds);

diff --git a/lib/util/index.js b/lib/util/index.js
@@ -70,9 +70,10 @@ function isRepositoryVersioned(repoInfo, repositoryId) {
 }
 
 function saveRepoToCache(repositoryId, type) {
-  const repoType = cache.get(repositoryId);
+   let subdomain = cds.context.user?.tokenInfo?.getPayload()?.ext_attr?.zdn;
+  const repoType = cache.get(repositoryId+"_"+subdomain);
   if (repoType === undefined) {
-    cache.set(repositoryId, type, 60 * 60 * 24 * 60);
+    cache.set((repositoryId+"_"+subdomain), type, 60 * 60 * 24 * 60);
   }
 }