Skip to content
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

Closed
michielbdejong opened this issue May 16, 2019 · 7 comments
Closed

What does write access to a container mean? #46

michielbdejong opened this issue May 16, 2019 · 7 comments

Comments

@michielbdejong
Copy link
Contributor

michielbdejong commented May 16, 2019

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 document
  • acl:Write means you can do 'DELETE' on that container

For 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

@RubenVerborgh
Copy link
Contributor

* `acl:Write` means you can do 'DELETE' on that container

Correction: means DELETE on resource inside of that container.

@michielbdejong
Copy link
Contributor Author

DELETE on resource inside of that container.

Oh, that's not how I read the current spec text. Where is that documented and/or implemented?

@michielbdejong
Copy link
Contributor Author

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?

@RubenVerborgh
Copy link
Contributor

"this should be a spec-level discussion" was just ignored there

Told you, the spec has massively been neglected, cfr. https://lists.w3.org/Archives/Public/public-solid/2019May/0015.html:

The "Solid spec" (however we frame it) wasn't intended to be above and beyond documenting a rough understanding and expectations to enabling the Solid ecosystem. So, nothing was written in stone and it only reflected what we arrived at with part implementations.

So definitely something that needs to be adjusted indeed.

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

The intention was to not do an rm -rf. In principle, if we are really strict with Cool URIs, a container deletion should result in a tombstone such that the same container can never be created again. But that obviously goes way too far for practical purposes.

I would argue that all deletions need to be manual. This does require visibility of the .acl file though.

In no circumstance should a user without Control permissions be able to cause the deletion of an .acl file, in whatever way.

@michielbdejong
Copy link
Contributor Author

OK, so conclusion:

@michielbdejong
Copy link
Contributor Author

Resolved, created #48 for the last point.

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

No branches or pull requests

2 participants