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

The scope of 503 may go beyond the request. See #99. #317

Closed
wants to merge 1 commit into from

Conversation

ioggstream
Copy link
Contributor

@ioggstream ioggstream commented Feb 10, 2020

This PR

Clarifies that 503 may apply not only to the given request.
This is because Retry-After states:

Retry-After indicates
how long the service is expected to be unavailable to the client.

Instead 503 states:

Retry-After [suggests an]
amount of time for the client to wait before retrying the request.

See: #99

Copy link
Member

@royfielding royfielding left a comment

Choose a reason for hiding this comment

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

I fail to see the value in this. The spec doesn't prevent further interpretation via future extensions, but 503 only refers to repeating the corresponding request (not all requests to the resource and not any request to the server). Suggesting that it might be one of those other things is not useful without some means of discovering which one.

@ioggstream
Copy link
Contributor Author

I'll prepare an I-D on that, if you think it's worth. Some ideas follows, wdyt?

An header

  • Backoff-Scope: header to be used to scope Retry-After or other headers (eg. ratelimit)

Link relations

This can be header-agile

Link: </books/>; rel="scope"; header="Retry-After"
Link: </books/>; rel="scope"; header="RateLimit-Limit"

@royfielding
Copy link
Member

I am not sure what you mean by those proposals, but a new header field like "RateLimit-Scope" (if it's about that) or "Retry-Scope" would make more sense.

@ioggstream
Copy link
Contributor Author

@martinthomson following Roy's suggestion I created a short extension draft https://ioggstream.github.io/draft-polli-retry-scope/draft-polli-retry-scope.html

Feedback welcome.

@mnot mnot mentioned this pull request Mar 19, 2020
@ioggstream
Copy link
Contributor Author

I linked the proposal in #99

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

Successfully merging this pull request may close these issues.

None yet

2 participants