-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
What does write access to a container mean? #46
Comments
Correction: means |
Oh, that's not how I read the current spec text. Where is that documented and/or implemented? |
Thanks for the link, slightly shocking that that remark from @dmitrizagidulin about "this should be a spec-level discussion" was just ignored there. I created #47 about it now, so we can discuss that there. That does bring me back to the original two points of this issue - when deleting a container, should the ACL doc be deleted, and should members and sub-members be deleted? |
Told you, the spec has massively been neglected, cfr. https://lists.w3.org/Archives/Public/public-solid/2019May/0015.html:
So definitely something that needs to be adjusted indeed.
The intention was to not do an I would argue that all deletions need to be manual. This does require visibility of the In no circumstance should a user without Control permissions be able to cause the deletion of an |
OK, so conclusion:
|
Resolved, created #48 for the last point. |
On a container, I think:
acl:Read
means you can list the container members ('GET')acl:Append
means you can add new resources into it ('POST')acl:Control
means you can create, update or delete that container's ACL documentacl:Write
means you can do 'DELETE' on that containerFor starters, it's not clear to me from the LDP spec whether deleting a container also deletes its members.
But apart from that, it's not clear to me from the WAC spec whether deleting a container also deletes its ACL document (especially since editing the ACL document without
acl:Control
access should be impossible, right?)Related to nodeSolidServer/node-solid-server#1179
The text was updated successfully, but these errors were encountered: