Merge pull request #1872 from nodeSolidServer/fix/remove-unused-resource-param

Fix/remove unused resource param
update to mashlib@2.2.2
update test replaced the stale TabulatorOutline marker with runDataBrowser

Author: bourgeoa

lib/header.mjs

@@ -119,8 +119,8 @@ export async function addPermissions (req, res, next) {
   const ldp = req.app.locals.ldp
   const resource = ldp.resourceMapper.resolveUrl(req.hostname, req.path)
   let [publicPerms, userPerms] = await Promise.all([
-    getPermissionsFor(acl, null, req),
-    getPermissionsFor(acl, session.userId, req)
+    getPermissionsFor(acl, null),
+    getPermissionsFor(acl, session.userId)
   ])
   if (resource.endsWith('.acl') && userPerms === '' && await ldp.isOwner(session.userId, req.hostname)) userPerms = 'control'
   debug.ACL(`Permissions on ${resource} for ${session.userId || '(none)'}: ${userPerms}`)
@@ -130,8 +130,8 @@ export async function addPermissions (req, res, next) {
   next()
 }
 
-// Gets the permissions string for the given user and resource
-async function getPermissionsFor (acl, user, req) {
+// Gets the permissions string for the given user
+async function getPermissionsFor (acl, user) {
   const accesses = MODES.map(mode => acl.can(user, mode))
   const allowed = await Promise.all(accesses)
   return PERMISSIONS.filter((mode, i) => allowed[i]).join(' ')