Change in ovirt-engine[master]: restapi: Add missing dependency to restapi-definition module

juan.hernandez at redhat.com juan.hernandez at redhat.com
Mon Oct 26 17:20:14 UTC 2015


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Add missing dependency to restapi-definition module
......................................................................


restapi: Add missing dependency to restapi-definition module

Add missing org.yaml.snakeyaml module to restapi-definition module.
When sending invalid json files to the server the server returned 500
instead of 400.

For instance the JsonExceptionMapper properly mapped
UnrecognizedPropertyException to return code 400, but the client
received 500 because constructing the response failed.

Change-Id: Ic2d6c8a74309d0ea1940714408202d096a8e1566
Bug-Url: https://bugzilla.redhat.com/1273932
Signed-off-by: Roman Mohr <rmohr at redhat.com>
---
M backend/manager/modules/restapi/interface/definition/src/main/modules/org/ovirt/engine/api/restapi-definition/main/module.xml
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Juan Hernandez: Looks good to me, approved
  Roman Mohr: Verified
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/47584
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic2d6c8a74309d0ea1940714408202d096a8e1566
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list