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

UC3 Inheritance #216

Merged
merged 37 commits into from
Jul 21, 2021
Merged
Changes from 1 commit
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
60313d0
UC 3 Inheritance
matthieubosquet May 26, 2021
7c7119f
Update uc-3-inheritance.md
matthieubosquet May 26, 2021
79a5139
Update uc-3-inheritance.md
matthieubosquet May 26, 2021
a760d8a
Update uc-3-inheritance.md
matthieubosquet May 26, 2021
2517390
get closer to UCR example, use relative URLs
bblfish Jun 2, 2021
ed285af
Align indenting
matthieubosquet Jun 4, 2021
1f7118c
Clarify wac group access
matthieubosquet Jun 9, 2021
1a5525d
Add missing prefix and clarify acl link
matthieubosquet Jun 9, 2021
52d9040
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jun 9, 2021
8872afd
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jun 9, 2021
45f90d6
Revert to relative container <.> syntax
matthieubosquet Jun 9, 2021
c483d59
Add an example resource hierarchy for ACP
matthieubosquet Jun 9, 2021
009f27f
Update proposals/evaluation/uc-3-inheritance.md
bblfish Jun 11, 2021
8bec8b8
Update proposals/evaluation/uc-3-inheritance.md
bblfish Jun 11, 2021
8116d93
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jun 15, 2021
4ef2fd1
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jun 15, 2021
5f6c351
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jun 15, 2021
9423e9d
added wac+:imports example
bblfish Jun 15, 2021
4905dea
Update proposals/evaluation/uc-3-inheritance.md
bblfish Jun 15, 2021
d68bc80
Fix use case, remove redundant text.
matthieubosquet Jun 30, 2021
2ead6fd
Add WAC effective resource and ACP modify
matthieubosquet Jul 1, 2021
f11ad58
Document structure fix
matthieubosquet Jul 1, 2021
563d705
Naming consistency
matthieubosquet Jul 1, 2021
c4fb0e8
Use same spacing everywhere
matthieubosquet Jul 2, 2021
f6db265
Ammend one proposed solution as not effective
matthieubosquet Jul 7, 2021
89ae405
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jul 7, 2021
3e70481
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jul 7, 2021
204023c
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jul 7, 2021
ced00e2
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jul 7, 2021
e5b5ffe
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jul 7, 2021
3fdca97
added the client point of view
bblfish Jul 14, 2021
f718fcf
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jul 19, 2021
05f66ff
Update proposals/evaluation/uc-3-inheritance.md
matthieubosquet Jul 19, 2021
d0436c2
Update proposals/evaluation/uc-3-inheritance.md
bblfish Jul 21, 2021
1af5c54
minor clarification
bblfish Jul 21, 2021
d853020
Merge branch 'uc3-inheritance' of github.com:bblfish/authorization-pa…
bblfish Jul 21, 2021
0fc465b
Add effective ACR explanation
matthieubosquet Jul 21, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 9 additions & 2 deletions proposals/evaluation/uc-3-inheritance.md
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,11 @@ To change the access rules to the `</weekly-status/2021-04-28/>` collection Bob
acl:mode acl:Read, acl:Write .
```

We could also use WAC's effective access control resource discovery mechanism and augment the content of `</weekly-status/.acl>`.
### WAC+ relaxing acl:default

As per the [ACL ontology definition as of July 2021](https://github.com/solid/authorization-panel/pull/216#discussion_r665338497), the `acl:default` predicate is only effective in statements where the current container is the object, that is, the resource `</weekly-status/.acl>` which is the direct effective access control list of `</weekly-status/>` can only use that directory as target in statements using `acl:default`.
matthieubosquet marked this conversation as resolved.
Show resolved Hide resolved

However, if WAC's use of `acl:default` were to be relaxed as described in [issue 191](https://github.com/solid/authorization-panel/issues/191), then one could rely on the effective access control resource discovery mechanism and augment the content of `</weekly-status/.acl>`:

```Turtle
# Resource: </weekly-status/.acl>
Expand All @@ -164,7 +168,10 @@ We could also use WAC's effective access control resource discovery mechanism an
acl:mode acl:Read, acl:Write .
```

### WAC+ac:imports
See also: https://github.com/solid/authorization-panel/pull/216#discussion_r665230245


### WAC+ ac:imports

[WAC+:imports](https://github.com/solid/authorization-panel/issues/210) works just as WAC does above with `:default` working as shown. The advantage of `ac:imports` is that the resource `</weekly-status/2021-04-28/.ac>` need only be set to contain:

Expand Down