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.

Sign up for GitHub

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

Jump to bottom

Reading and Writing resources #188

Merged
merged 49 commits into from
Aug 3, 2020
Merged

Reading and Writing resources #188

merged 49 commits into from
Aug 3, 2020

Conversation

csarven
Copy link
Member

@csarven csarven commented Jul 8, 2020 •
edited
Loading

In response to the following (and possibly other) issues on how read, append, write and delete operations work in context of Web Access Control:

@csarven
Init delete operation in context of WAC
a30bb1e
@csarven
Reject delete root container or its ACL
96264b8
@csarven
Add acl:Write requirement to delete a resource
a5cc118
@csarven
Add removal of containment triple on container
d28ec63
@csarven
Add acl:Control requirement to delete an ACL
bd435d3
@csarven
Add rules to delete a container
ad120fd
@csarven
Add option to omit DELETE in Allow
342febe
@csarven
Init non-normative for delete's clean-up
36c0d8a
@csarven
Add common 404/410 response to GET after DELETE
b46b54a
@csarven
Note reinstating resources and ACL change
918484a
@csarven
Add todo related to delete
b7426e9
@csarven
Specify delete container without recursion
c11eab4
@csarven
Minor: spacing
682d354
@csarven
Move Deleting Resources under Reading and Writing Resources
8d41201
@csarven
Init Writing resources; PUT POST PATCH
05324d4
@csarven
Move creating intermediate containers
0952b6f
@csarven
Move POST interactions
0e48e55
@csarven
Move read-write
314b9d2
@csarven
POST to update reosurce
d8f6d98
@csarven
PUT PATCH targeting auxiliary resource
41c8dc3
@csarven
Disallow containment triples updates
1dd144c
@csarven
If-None-Match against updating
5464eca
@csarven
If-Match for updating
af15522
@csarven
Add requirements to update ACL
4a4eaeb
@csarven
Add non-normative for POST+Slug
0ac9ae9
@csarven
Init Reading Resources; GET HEAD OPTIONS
f208eb3
@csarven
Add Allow in context of WAC
d2831db
@csarven
Add Accept-Patch Accept-Post with Allow
da1818d
@csarven
Add OPTIONS * and Accept-Patch Accept-Post
21e5c39
@csarven csarven mentioned this pull request Jul 9, 2020
Closed
justinwb
justinwb approved these changes Jul 17, 2020
View reviewed changes
Copy link
Member

@justinwb justinwb left a comment

Choose a reason for hiding this comment

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

LGTM ✅

acoburn
acoburn reviewed Jul 19, 2020
View reviewed changes
main/resource-access.bs Outdated Show resolved Hide resolved
main/resource-access.bs Outdated Show resolved Hide resolved
@csarven csarven mentioned this pull request Jul 27, 2020
Closed
dmitrizagidulin
dmitrizagidulin approved these changes Jul 27, 2020
View reviewed changes
Copy link
Member

@dmitrizagidulin dmitrizagidulin left a comment

Choose a reason for hiding this comment

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

👍 looks good.

@csarven csarven merged commit c7fdf49 into master Aug 3, 2020
@csarven csarven mentioned this pull request Sep 11, 2020
Closed
@csarven csarven mentioned this pull request Oct 30, 2020
Closed
@csarven csarven deleted the feature/write branch December 4, 2020 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@acoburn acoburn acoburn approved these changes

@RubenVerborgh RubenVerborgh RubenVerborgh approved these changes

@dmitrizagidulin dmitrizagidulin dmitrizagidulin approved these changes

@justinwb justinwb justinwb approved these changes

Assignees
No one assigned
Labels
topic: resource access topic: resource life cycle
Projects
Specification
Status: Done
Milestone
~First Public Working Draft
Development

Successfully merging this pull request may close these issues.
5 participants
@csarven @justinwb @RubenVerborgh @dmitrizagidulin @acoburn