Improving variable naming in allow handler

Author: rubensworks

lib/handlers/allow.js

@@ -8,7 +8,7 @@ const fs = require('fs')
 const { promisify } = require('util')
 const HTTPError = require('../http-error')
 
-function allow (mode, testDirectory) {
+function allow (mode, checkPermissionsForDirectory) {
   return async function allowHandler (req, res, next) {
     const ldp = req.app.locals.ldp || {}
     if (!ldp.webid) {
@@ -20,31 +20,31 @@ function allow (mode, testDirectory) {
 
     // Determine the actual path of the request
     // (This is used as an ugly hack to check the ACL status of other resources.)
-    let reqPath = res && res.locals && res.locals.path
+    let resourcePath = res && res.locals && res.locals.path
       ? res.locals.path
       : req.path
 
-    // If a relativePath has been provided, check permissions based on that
-    if (testDirectory) {
-      reqPath = path.dirname(reqPath)
+    // Check permissions of the directory instead of the file itself.
+    if (checkPermissionsForDirectory) {
+      resourcePath = path.dirname(resourcePath)
     }
 
     // Check whether the resource exists
     let stat
     try {
-      const ret = await ldp.exists(req.hostname, reqPath)
+      const ret = await ldp.exists(req.hostname, resourcePath)
       stat = ret.stream
     } catch (err) {
       stat = null
     }
 
     // Ensure directories always end in a slash
-    if (!reqPath.endsWith('/') && stat && stat.isDirectory()) {
-      reqPath += '/'
+    if (!resourcePath.endsWith('/') && stat && stat.isDirectory()) {
+      resourcePath += '/'
     }
 
     // Obtain and store the ACL of the requested resource
-    req.acl = new ACL(rootUrl + reqPath, {
+    req.acl = new ACL(rootUrl + resourcePath, {
       agentOrigin: req.get('origin'),
       // host: req.get('host'),
       fetch: fetchFromLdp(ldp.resourceMapper),