Change in ovirt-engine[master]: restapi: Validate boolean values

juan.hernandez at redhat.com juan.hernandez at redhat.com
Wed Jun 4 13:58:27 UTC 2014


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Validate boolean values
......................................................................


restapi: Validate boolean values

JAXB doesn't validate boolean values, anything that isn't the string
"true" is interpreted as false. This patch introduces a custom parser
for the boolean type that only accepts the strings "true" and "false",
any other thing will be rejected.

Change-Id: Ibb3558185a46869a73115b936183cb22007c99e9
Bug-Url: https://bugzilla.redhat.com/1063876
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/BooleanParser.java
M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xjb
2 files changed, 26 insertions(+), 0 deletions(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/27925
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibb3558185a46869a73115b936183cb22007c99e9
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list