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@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 Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/38055 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1330df35a054713d2e42c00bf70ea52ff0b49021 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alon Bar-Lev <alonbl@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Ravi Nori <rnori@redhat.com> Gerrit-Reviewer: automation@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server