Michael Pasternak has submitted this change and it was merged.
Change subject: restapi : Add Role with wrong permit id leads to 500 error(#908392)
......................................................................
restapi : Add Role with wrong permit id leads to 500 error(#908392)
Add Role with wrong permit id leads to internal server error.
This patch returns a proper error message when the permit id
is invalid.
Change-Id: I9adfda37e67d24f96933aa4eb8f79a8fdb416d62
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=908392
Signed-off-by: Ravi Nori <rnori(a)redhat.com>
---
M
backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendRolesResourceTest.java
M
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/PermitMapper.java
2 files changed, 30 insertions(+), 1 deletion(-)
Approvals:
Michael Pasternak: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/11824
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9adfda37e67d24f96933aa4eb8f79a8fdb416d62
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>