Say Unauthenticated for 401

Author: kjetilk

lib/acl-checker.js

@@ -69,9 +69,9 @@ class ACLChecker {
     const accessDenied = aclCheck.accessDenied(this.acl.graph, resource, directory, aclFile, agent, modes, origin, trustedOrigins)
     console.log('ACCESS DENIED', accessDenied, '\n\n')
     if (accessDenied && user) {
-      throw new HTTPError(403, `Access to ${this.resource} denied for ${user}`)
+      throw new HTTPError(403, accessDenied)
     } else if (accessDenied) {
-      throw new HTTPError(401, `Access to ${this.resource} requires authorization`)
+      throw new HTTPError(401, 'Unauthenticated')
     }
     return Promise.resolve(true)
   }

lib/http-error.js

@@ -1,6 +1,7 @@
 module.exports = HTTPError
 
 function HTTPError (status, message) {
+  console.log('DAAAAHUT: ', status, message)
   if (!(this instanceof HTTPError)) {
     return new HTTPError(status, message)
   }

test/integration/acl-oidc-test.js

@@ -247,7 +247,7 @@ describe('ACL with WebID+OIDC over HTTP', function () {
     })
   })
 
-  describe('Origin', function () {
+  describe.only('Origin', function () {
     before(function () {
       rm('/accounts-acl/tim.localhost/origin/test-folder/.acl')
     })
@@ -332,6 +332,7 @@ describe('ACL with WebID+OIDC over HTTP', function () {
         options.headers.origin = origin2
 
         request.head(options, function (error, response, body) {
+          console.log(response)
           assert.equal(error, null)
           assert.equal(response.statusCode, 403)
           assert.equal(response.statusMessage, 'Origin Unauthorized')