Skip to content
This repository has been archived by the owner on Apr 13, 2022. It is now read-only.

proposal for wac:defaultInclude relation for acl. #79

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bblfish
Copy link
Contributor

@bblfish bblfish commented Feb 1, 2016

Note: as this document is really a work in progress discussion space, it would probably have been better to place it in the wiki.

@bblfish bblfish force-pushed the patch-2 branch 2 times, most recently from e446f0a to 56e8a80 Compare February 1, 2016 12:38
@bblfish bblfish force-pushed the patch-2 branch 2 times, most recently from cb6dd72 to 5754325 Compare February 1, 2016 13:23
@@ -21,14 21,14 @@ In `monotonic`, ACL permissions are cumulative (inherited from the ancestors) an

#### Pro
- Not as fast as defaultForNew but can be
- Simple hierarchical permission (e.g. everything in `/shared` is shared)
- Simple hierarchical permission (e.g. everything in `/shared` is shared) (bblfish: does that mean that a resource cannot unlink from a previous permission?)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bblfish - can you do comments via the Github PR UI (mouse over the line number that you want to comment on, it turns into , which opens the comments textbox), instead of inline in the actual text?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes, had not thought of that.

- Can be fast as it only has to find one ACL file to give the permission it needs
- Monotonic: Once a user or any agent knows the ACL it can apply it as a rule. An ACL is a first class fact. It can be digitally signed, transported, and used to demand access at a later date, etc. Monotonicness is useful.

#### Cons
- It is slower than `defaultFor new`, but the search stops the moment it finds success.
- It can't have private subfolders within shared folders. Given that permissions cannot be reverted (with the current WAC specification), a subfolder cannot be private in a shared folder. This system is monotonic.
- User has to be aware of the permissions given to the parent folders
- User has to be aware of the permissions given to the parent folders
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not use wac:include?

I don't think I quite understand this section.
Note that it is possible to comment here, but I don't think it is possible to comment on the the main branch...

@bblfish
Copy link
Contributor Author

bblfish commented Feb 1, 2016

So it was decided that PRs are better than wikis.

Note: The section I added may be related to the previous points but I can't tell for sure, as there are a lot of words that are not defined clearly enough for me. Hopefully this can be clarified through a discussion.

@kjetilk kjetilk added this to the Spec Pull Requests milestone Apr 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants