Skip to content

Commit 2ba5a9b

Browse files
author
NaderMohamed325
committed
Remove unused resource parameter from getPermissionsFor
1 parent 2caddca commit 2ba5a9b

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

lib/header.mjs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -119,8 +119,8 @@ export async function addPermissions (req, res, next) {
119119
const ldp = req.app.locals.ldp
120120
const resource = ldp.resourceMapper.resolveUrl(req.hostname, req.path)
121121
let [publicPerms, userPerms] = await Promise.all([
122-
getPermissionsFor(acl, null, req),
123-
getPermissionsFor(acl, session.userId, req)
122+
getPermissionsFor(acl, null),
123+
getPermissionsFor(acl, session.userId)
124124
])
125125
if (resource.endsWith('.acl') && userPerms === '' && await ldp.isOwner(session.userId, req.hostname)) userPerms = 'control'
126126
debug.ACL(`Permissions on ${resource} for ${session.userId || '(none)'}: ${userPerms}`)
@@ -131,7 +131,7 @@ export async function addPermissions (req, res, next) {
131131
}
132132

133133
// Gets the permissions string for the given user and resource
134-
async function getPermissionsFor (acl, user, req) {
134+
async function getPermissionsFor (acl, user) {
135135
const accesses = MODES.map(mode => acl.can(user, mode))
136136
const allowed = await Promise.all(accesses)
137137
return PERMISSIONS.filter((mode, i) => allowed[i]).join(' ')

0 commit comments

Comments
 (0)