Change in ovirt-engine[master]: restapi : Add Role with wrong permit id leads to 500 error(#...

mpastern at redhat.com mpastern at redhat.com
Mon Feb 11 10:07:19 UTC 2013


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 at 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 at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>



More information about the Engine-commits mailing list