nodeSolidServer / node-solid-server

Solid server on top of the file-system in NodeJS
https://solidproject.org/for-developers/pod-server
Other
1.78k stars 299 forks source link

WAC-Allow doesn't seem to depend on resource #973

Open kjetilk opened 5 years ago

kjetilk commented 5 years ago

Looking at the code in https://github.com/solid/node-solid-server/blob/adb0691a787d65e41cfd9e32d4496d8a0b4f98ed/lib/header.js#L132 it doesn't seem like #246 was implemented with resource, because that parameter doesn't seem to be used.

RubenVerborgh commented 5 years ago

Indeed, resource is implicitly in acl.

kjetilk commented 5 years ago

Perhaps you want to take this little trivial thing @jaxoncreed , just a cleanup?