Skip to content
This repository has been archived by the owner on May 25, 2021. It is now read-only.

ACLs are created incorrectly on the Pod by the App #275

Open
jholleran opened this issue Nov 21, 2019 · 2 comments
Open

ACLs are created incorrectly on the Pod by the App #275

jholleran opened this issue Nov 21, 2019 · 2 comments

Comments

@jholleran
Copy link

Describe the bug
ACLs are created on the Pod with an explicit .acl extension. Although this will work for NSS it can't assume that it will be that prefix for every LDP Resource Server. Instead, it needs to read the Link header rel-"acl" on the created resource and use this URI to set the ACL.

To Reproduce
Steps to reproduce the behavior:

  1. Run ESS a the Resource Server
  2. Login an start a TicTacToe games with another Player
  3. See resources created under public/games/tictactoe/

Expected behavior
ACLs should be created with the correct URI for the server

Screenshots
Here is a GET request to the parent container which is treating it as a resource and not an ACL:
image

Desktop (please complete the following information):
N/A

Smartphone (please complete the following information):
N/A

Additional context
Here is the Link headers of that resource which gives the correct ACL URI that needs to be used:
image

@michielbdejong
Copy link
Contributor

@james-martin-jd any thoughts?

Maybe https://github.com/Otto-AA/solid-acl-utils/blob/master/src/AclApi.ts is helpful for this?

Here's also a snippet for how to edit ACL docs using TripleDoc:
https://gist.github.com/michielbdejong/858f83d6cf6ffadd71251b79db82a98f

@james-martin-jd
Copy link
Contributor

We plan to implement this change soon, it unfortunately didn't make it into the 0.7.0 release but we have an internal ticket to track it as well.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants