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

Adding a rel='deletes' link #436

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

qmachard
Copy link
Contributor

In some case, a POST or PUT call can delete some resources.

The problem is, when a HTTP method returns a Link header that contains a
rel="invalidates", then this can signal a client that the cache for the
collection is no longer fresh, but send event to refresh a deleted resources. (cf. #152)

My solution is adding new rel option called deletes.

Link: </foo>; rel="deletes"

When a HTTP method returns a Link header that contains a
rel="deletes", then it means that the HTTP request that was recently
performed has caused other deleted resources.

@qmachard
Copy link
Contributor Author

qmachard commented Jun 8, 2022

Hi @evert, what do you think about this PR ?

@evert
Copy link
Collaborator

evert commented Jun 8, 2022

I forgot to comment on this originally, but probably my only concern is that this is not based on any published or draft standard.

Generally I'm not too concerned introducing my own rels in private projects, but given the more public nature of this one I wonder if it wouldn't be better if we wrote a sort of spec for this somewhere.

The idea is great, and falls right in line with the general architecture and philosophy. Excellent PR as well.

I can be convinced either way though, but curious what you think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants