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

Stop using 501 http status #51

Open
karlcz opened this issue Feb 26, 2019 · 0 comments
Open

Stop using 501 http status #51

karlcz opened this issue Feb 26, 2019 · 0 comments

Comments

@karlcz
Copy link
Contributor

karlcz commented Feb 26, 2019

There is some misguided use of 501 Not Implemented status to complain when the client asks for range requests in scenarios which are not supported. I think there are separate cases to consider:

  1. We don't want to support it: just return 200 OK with full response
  2. It is malformed: return 400 Bad Request?
  3. It is unsatisfiable: return 416 Requested Range Not Satisfiable

none of these are particularly relevant to well behaved clients, but it seems we have encountered bad clients who send Range headers on resources where we do not advertise support via Accept-Range.

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

1 participant