support extension usage for relationship links #1650
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The current wording for updating relationships is too restrictive. Most noticeable it limits usage of extensions for endpoints used as relationship links.
For example, it forbids servers to support the atomic operations extensions on endpoints used for relationship links.
This conflicts with atomic operations extension. The atomic operations extension specifies that a client may send a
POST
request withatomic:operations
top-level member. Thedata
top-level member must not be present. Supporting such a request on an endpoint used for relationship links as well would violate the specification if taking it by word.This merge request changes the wording to allow extensions like atomic operations on these endpoints.
It does so by changing the wording to focus on the intend. It specifies how a request for updating relationships must look like. But does not put any constraints on other requests - even if using the same endpoint.
This aligns updating relationships with existing wording for creating, updating and deleting a resource.