Change in ovirt-engine[master]: aaa: always set principal when fetching principal record

oourfali at redhat.com oourfali at redhat.com
Mon Feb 23 09:59:36 UTC 2015


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 at 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 at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali 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