Moti Asayag has submitted this change and it was merged. Change subject: extapi: authn: logout should accept auth record ...................................................................... extapi: authn: logout should accept auth record this must have been an error in documentation, logout should accept an auth record, from which it can extract principal and other information such as session id. we can fix this now as there is no known usage of logout so far. Change-Id: I8616bebdcdb620e8d38c171e07af8c93edfd9b17 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com&gt; --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/LogoutSessionCommand.java M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/aaa/Authn.java 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43691 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8616bebdcdb620e8d38c171e07af8c93edfd9b17 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com&gt; Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org