Adding caching of requests to acl-checker

Author: megoth

lib/acl-checker.js

@@ -24,6 +24,7 @@ class ACLChecker {
     this.suffix = options.suffix || DEFAULT_ACL_SUFFIX
     this.aclCached = {}
     this.messagesCached = {}
+    this.requests = {}
   }
 
   // Returns a fulfilled promise when the user can access the resource
@@ -139,7 +140,8 @@ class ACLChecker {
       const acl = possibleACLs.shift()
       let graph
       try {
-        graph = await this.fetch(acl)
+        this.requests[acl] = this.requests[acl] || this.fetch(acl)
+        graph = await this.requests[acl]
       } catch (err) {
         if (err && (err.code === 'ENOENT' || err.status === 404)) {
           isContainer = true
@@ -157,8 +159,13 @@ class ACLChecker {
       throw new HTTPError(500, `No ACL found for ${resource}, searched in \n- ${acls.join('\n- ')}`)
     }
     console.log('>>>> GRAPH WITHOUT GROUPS', returnAcl.graph.length)
-    const groupUrls = returnAcl.graph.statementsMatching(null, ACL('agentGroup'), null).map(node => node.object.value.split('#')[0])
-    await Promise.all(groupUrls.map(groupUrl => this.fetch(groupUrl, returnAcl.graph)))
+    const groupUrls = returnAcl.graph
+      .statementsMatching(null, ACL('agentGroup'), null)
+      .map(node => node.object.value)
+    await Promise.all(groupUrls.map(groupUrl => {
+      this.requests[groupUrl] = this.requests[groupUrl] || this.fetch(groupUrl, returnAcl.graph)
+      return this.requests[groupUrl]
+    }))
     console.log('>>>> GRAPH WITH GROUPS', returnAcl.graph.length)
 
     return returnAcl