You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think this library is too strict and should not assume there's a body in case of error 4xx. the rfc says SHOULD contain a body, and not MUST. Which by the definition of SHOULD in rfc2119 means it's only recommended.
Thoughts?
10.4 Client Error 4xx
"the server SHOULD include an entity containing an explanation of the error situation, and whether it is a temporary or permanent condition. These status codes are applicable to any request method. User agents SHOULD display any included entity to the user."
You are absolutely right. I designed this library for a specific project in which there always was a body set in case of a 4xx error. I'm going to change it in 1.4.0 release or just feel free to create a pull request.
I think this library is too strict and should not assume there's a body in case of error 4xx. the rfc says SHOULD contain a body, and not MUST. Which by the definition of SHOULD in rfc2119 means it's only recommended.
Thoughts?
10.4 Client Error 4xx
"the server SHOULD include an entity containing an explanation of the error situation, and whether it is a temporary or permanent condition. These status codes are applicable to any request method. User agents SHOULD display any included entity to the user."
https://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html
The text was updated successfully, but these errors were encountered: