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

Relax requirement to use HTTP 501 #415

Closed
m-mohr opened this issue Oct 11, 2021 · 0 comments · Fixed by #418
Closed

Relax requirement to use HTTP 501 #415

m-mohr opened this issue Oct 11, 2021 · 0 comments · Fixed by #418
Assignees
Labels
breaking Breaking changes, requires a major-version (2.0.0 for example)
Milestone

Comments

@m-mohr
Copy link
Member

m-mohr commented Oct 11, 2021

The API currently says:

404 Not Found: The resource specified by the path does not exist, i.e. one of the resources belonging to the specified identifiers are not available at the back-end. Note: Unsupported endpoints MUST use HTTP status code 501.

This is pretty wild as the default for unsupported endpoints in many environments is 404. Thus you'd need to implement special handling to specifically all endpoints you don't want to implement (in the future). None of the clients (AFAIK) and also none of the back-ends really makes use of the 501, so we could also just weaken the requirement and say:

Note: Unsupported endpoints MAY use HTTP status code 501.

m-mohr added a commit that referenced this issue Oct 19, 2021
@m-mohr m-mohr linked a pull request Oct 19, 2021 that will close this issue
3 tasks
@m-mohr m-mohr self-assigned this Oct 19, 2021
@m-mohr m-mohr added this to the 1.2.0 milestone Oct 19, 2021
@m-mohr m-mohr added the breaking Breaking changes, requires a major-version (2.0.0 for example) label Oct 19, 2021
@m-mohr m-mohr closed this as completed Oct 22, 2021
This was referenced May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking changes, requires a major-version (2.0.0 for example)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant