-
Notifications
You must be signed in to change notification settings - Fork 887
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
Additional HTTP Status Codes #1372
Conversation
+1 LGTM |
Additional HTTP Status Codes
Thanks for the new codes! |
Any idea when 1.5.3 will come out? I was actually just needing this feature myself. |
As a new feature I hadn't intended to put this in the 1.5 series. |
Ok. thanks for the fast reply. I'll just have our deployscript apply this as a patch for now. |
@jvanasco Unless you specifically need the Exceptions, you can already set these status codes on a response and WebOb will correctly send out the correct information. |
Thanks. Unfortunately, I needed to raise an Exception because the existing code relies on I should bring up an idea I had though for making the exceptions a bit more extensible. I'll propose that in a new ticket. |
HTTP Status codes from RFC 6585,
Especially 429 - Too Many Requests is pretty useful for APIs, I think it is worth including.
I did not implement 511 - Network Authentication Required because it is