Skip to content

Commit 0ece321

Browse files
committed
Some optimizations
1 parent 1998a5b commit 0ece321

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

lib/acl-checker.js

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,10 @@ class ACLChecker {
1616
constructor (resource, options = {}) {
1717
this.resource = resource
1818
this.resourceUrl = new URL(resource)
19-
this.agentOrigin = options.agentOrigin
19+
this.agentOrigin = options.strictOrigin && options.agentOrigin ? rdf.sym(options.agentOrigin) : null
2020
this.fetch = options.fetch
2121
this.fetchGraph = options.fetchGraph
22-
this.strictOrigin = options.strictOrigin
23-
this.trustedOrigins = options.trustedOrigins
22+
this.trustedOrigins = options.strictOrigin && options.trustedOrigins ? options.trustedOrigins.map(trustedOrigin => rdf.sym(trustedOrigin)) : null
2423
this.suffix = options.suffix || DEFAULT_ACL_SUFFIX
2524
this.aclCached = {}
2625
this.messagesCached = {}
@@ -56,8 +55,8 @@ class ACLChecker {
5655
const aclFile = rdf.sym(acl.acl)
5756
const agent = user ? rdf.sym(user) : null
5857
const modes = [ACL(mode)]
59-
const agentOrigin = this.strictOrigin && this.agentOrigin ? rdf.sym(this.agentOrigin) : null
60-
const trustedOrigins = this.strictOrigin && this.trustedOrigins ? this.trustedOrigins.map(trustedOrigin => rdf.sym(trustedOrigin)) : null
58+
const agentOrigin = this.agentOrigin
59+
const trustedOrigins = this.trustedOrigins
6160
const accessDenied = aclCheck.accessDenied(acl.graph, resource, directory, aclFile, agent, modes, agentOrigin, trustedOrigins)
6261

6362
if (accessDenied && user) {

0 commit comments

Comments
 (0)