openstack/nova
Code Issues Proposed changes

Add a note about the 400 response not requiring a microversion

Browse Source 
As the API validation code always return 400 when request with
invalid json body, so 400 is an available status code for all the
Nova API. Then no microversion bump required for fixing 500 to 400.

Change-Id: I0454e95cd0655c910fad8b39ead2b5edab216b68
Related-Bug: 1471239
This commit is contained in:
He Jie Xu
2015-08-27 16:12:31 +08:00
parent ed647de464
commit 79a9ca6da3
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
doc/source/api_microversion_dev.rst
View File

@@ -65,7 +65,7 @@ changed. The user contract covers many kinds of information such as:
Example: an API previously could return 200, 400, 403, 404 and the
change would make the API now also be allowed to return 409.
See [#f2]_ for the 403 case.
See [#f2]_ for the 400 and 403 cases.
- changing a status code on a particular response
@@ -154,9 +154,10 @@ we need a microversion".
microversion is probably needed.
.. [#f2] The exception to not needing a microversion when returning a
previously unspecified error code is the 403 case. This is considered OK to
return even if previously unspecified in the code since it's implied given
keystone authentication can fail with a 403.
previously unspecified error code is the 400 and 403 cases. This is
considered OK to return even if previously unspecified in the code since
it's implied given keystone authentication can fail with a 403 and API
validation can fail with a 400 for invalid json request body.
In Code