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

Clarify that PUT for updating in RSR means a complete replacement #177

Closed
xmlgrrl opened this issue Jul 20, 2015 · 1 comment
Closed

Clarify that PUT for updating in RSR means a complete replacement #177

xmlgrrl opened this issue Jul 20, 2015 · 1 comment
Labels
critical rsrc-reg Related to resource registration (or the original UMA1 resource reg spec) V1.0.1
Milestone

Comments

@xmlgrrl
Copy link

xmlgrrl commented Jul 20, 2015

The Update Resource Set Description operation (RSR Sec 2.3: https://docs.kantarainitiative.org/uma/rec-oauth-resource-reg.html#rfc.section.2.3.3, also referenced in the worked-example appendix) uses a PUT, and the intent was for it to be a complete replacement per our Design Principle 4 about RESTfulness (http://kantarainitiative.org/confluence/display/uma/UMA+Requirements). But it's technically possible for a PUT not to be a complete replacement, and for interoperability's sake in terms of authorization server functionality we should make this clear.

@xmlgrrl xmlgrrl added the rsrc-reg Related to resource registration (or the original UMA1 resource reg spec) label Jul 20, 2015
@mmachulak
Copy link

Agreed, although after "Form of an update request, with an access token in the header:" this is clear that the body request has to contain a body of the full resource set description - see "(body contains JSON resource set description to be updated)"

@xmlgrrl xmlgrrl added this to the V1.0.1 milestone Jul 31, 2015
@xmlgrrl xmlgrrl added the V1.0.1 label Aug 13, 2015
xmlgrrl added a commit that referenced this issue Aug 19, 2015
Clarified that the Update method using PUT requires a complete
replacement of the body.
@xmlgrrl xmlgrrl closed this as completed Aug 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
critical rsrc-reg Related to resource registration (or the original UMA1 resource reg spec) V1.0.1
Projects
None yet
Development

No branches or pull requests

2 participants