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

Usage of HTTP 501 vs 405 for missing methods. #101

Closed
evert opened this issue Oct 14, 2012 · 3 comments
Closed

Usage of HTTP 501 vs 405 for missing methods. #101

evert opened this issue Oct 14, 2012 · 3 comments
Labels

Comments

@evert
Copy link
Member

evert commented Oct 14, 2012

Original author: evert...@gmail.com (February 23, 2010 08:19:47)

Basically, the rule is:

HTTP 501 for methods we don't recognize
HTTP 405 for methods we don't recognize for one particular resource.

We are using 501 in a couple of places where 405 is more appropriate

  • Locks_Plugin (resource does not support locks)
  • GET/HEAD on Collection resource

The problem with using 405 for this instead, is the following clause:

The response MUST include an
Allow header containing a list of valid methods for the requested
resource.

This constraint has actually been violated in 6 other instances (just do a
grep).

Original issue: http://code.google.com/p/sabredav/issues/detail?id=20

@evert evert closed this as completed Oct 14, 2012
@evert
Copy link
Member Author

evert commented Oct 14, 2012

From evert...@gmail.com on April 23, 2010 12:09:57:
Fixed on mercurial.

@evert
Copy link
Member Author

evert commented Feb 6, 2020

You found a 10 year old ticket =)

501 is a nice indicator that a developer just hasn't done something, but unfortunately it's also technically incorrect: https://evertpot.com/http/501-not-implemented

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

No branches or pull requests

2 participants