openstack/placement
Code Issues Proposed changes

Placement api: set custom json_error_formatter in root

Browse Source 
Change decorator for resource based actions in root
handler to wsgi_wrapper.PlacementWsgify

This is a newly introduced wrapper class around webob.dec.wsgify
to set json formatter in case of webob exceptions.

Change-Id: I847345f7f15a46930faf8793baf801beba98d368
Closes-Bug: #1635182
This commit is contained in:
Pushkar Umaranikar 2017-02-13 22:22:24 +00:00
parent a2b85463b4
commit 77133bbf50
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
nova/api/openstack/placement/handlers/root.py
View File

@ -13,13 +13,13 @@
from oslo_serialization import jsonutils from oslo_serialization import jsonutils
from oslo_utils import encodeutils from oslo_utils import encodeutils
import webob
from nova.api.openstack.placement import microversion from nova.api.openstack.placement import microversion
from nova.api.openstack.placement import wsgi_wrapper
@webob.dec.wsgify @wsgi_wrapper.PlacementWsgify
def home(req): def home(req):
min_version = microversion.min_version_string() min_version = microversion.min_version_string()
max_version = microversion.max_version_string() max_version = microversion.max_version_string()

6
nova/tests/functional/api/openstack/placement/gabbits/microversion.yaml
View File

@ -36,6 +36,8 @@ tests:
$.errors.[0].min_version: /^\d+\.\d+$/ $.errors.[0].min_version: /^\d+\.\d+$/
response_strings: response_strings:
- "Unacceptable version header: 0.5" - "Unacceptable version header: 0.5"
response_json_paths:
$.errors[0].title: Not Acceptable
- name: latest microversion is 1.4 - name: latest microversion is 1.4
GET: / GET: /
@ -63,6 +65,8 @@ tests:
status: 400 status: 400
response_strings: response_strings:
- "invalid version string: pony.horse" - "invalid version string: pony.horse"
response_json_paths:
$.errors[0].title: Bad Request
- name: bad format multidot raises 400 - name: bad format multidot raises 400
GET: / GET: /
@ -71,6 +75,8 @@ tests:
status: 400 status: 400
response_strings: response_strings:
- "invalid version string: 1.2.3" - "invalid version string: 1.2.3"
response_json_paths:
$.errors[0].title: Bad Request
- name: error in application produces microversion headers - name: error in application produces microversion headers
desc: we do not want xml desc: we do not want xml