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

Expand on async processing #36

Open
duglin opened this issue Oct 29, 2018 · 1 comment
Open

Expand on async processing #36

duglin opened this issue Oct 29, 2018 · 1 comment

Comments

@duglin
Copy link

duglin commented Oct 29, 2018

I think more is needed w.r.t. describing how async works. I would prefer a model like:

  • request is sent to the server
  • 202 Accepted is returned + Location header for the client to check status
    • even if the Location is the same URL as the resource itself
  • in the case of a failed create(), the resource at the Location URL still exists but with the failed status and error message. How long it lives for is an impl detail. But this is also a good reason to NOT let the client pass in the ID and use it as part of the URL of resource. Let each create result in a new resource.
  • in the case of an async delete(), a GET on the resource will always return at least the status section until the resource is gone - then a 404 would be returned.
@KirilKabakchiev
Copy link
Member

I added some more details in #31 while also restructuring the spec a bit. Let me know 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

No branches or pull requests

2 participants