Oved Ourfali has submitted this change and it was merged.
Change subject: aaa: always set principal when fetching principal record
......................................................................
aaa: always set principal when fetching principal record
although information is available in AuthRecord, and extensions should
fall back into optional Principal anyway (as already do), it would be
nicer to provide both Principal and AuthRecord if available to simplify
extension implementation.
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1192791
Change-Id: I1330df35a054713d2e42c00bf70ea52ff0b49021
Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com>
---
M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java
1 file changed, 26 insertions(+), 10 deletions(-)
Approvals:
Alon Bar-Lev: Verified
Ravi Nori: Looks good to me, but someone else must approve
Oved Ourfali: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/37799
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1330df35a054713d2e42c00bf70ea52ff0b49021
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server