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

405 for "other" methods? #117

Closed
kjetilk opened this issue Nov 14, 2019 · 2 comments
Closed

405 for "other" methods? #117

kjetilk opened this issue Nov 14, 2019 · 2 comments

Comments

@kjetilk
Copy link
Member

kjetilk commented Nov 14, 2019

I just created a test with the following request:

:test_bob_r_o_req a http:RequestMessage ;
    http:method "DAHU" ;
    http:requestURI </test-auth/alice_share_bob_r.txt> .

NSS returns 400 while I expected 405.

I think it is good practice to return a 405 for methods you do not implement, since it tends to be trivial to do and affords the client with more details to adjust its interactions. The RFC is more relaxed on this topic than I tend to be though. The question is if we should say something normative about it, and if so, should that normative statement be a MUST?

@csarven
Copy link
Member

csarven commented Nov 14, 2019

For that test, 405 is precise whereas 400 would be vague. I find 405 to be more useful for clients and don't particularly see why a server would choose to return a 400 especially if the issue is strictly about the method not being available on a given resource. There is nothing wrong, faulty, unsound with the request itself. If there are other dimensions that's taken into account, 400 may be more meaningful but that'd be a different test.

@kjetilk kjetilk added this to the June 19th milestone Nov 14, 2019
@kjetilk
Copy link
Member Author

kjetilk commented Nov 14, 2019

Good, so we agree!

Not a big issue, I'll set it on the June milestone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

2 participants