Change in ovirt-engine[master]: aaa: Running the command logon on the VM via the REST failed

Code Review gerrit at ovirt.org
Fri Jun 23 19:27:38 UTC 2017


>From Martin Peřina <mperina at redhat.com>:

Martin Peřina has submitted this change and it was merged.

Change subject: aaa: Running the command logon on the VM via the REST failed
......................................................................


aaa: Running the command logon on the VM via the REST failed

Passwords should be encrypted using engine
pki certificate and not using client
cetificate. Restapi does not send client
id so using engine pki cert SSO can still
save encrypted password in SsoSession.

The saved password is used to execute VmLogon
command.

Change-Id: I049acc6ab02e173ae52aa12133c0a13eb1ec223b
Bug-Url: https://bugzilla.redhat.com/1463634
Signed-off-by: Ravi Nori <rnori at redhat.com>
---
M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/context/SsoContextListener.java
M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/SsoContext.java
M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/SsoLocalConfig.java
M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/SsoUtils.java
4 files changed, 36 insertions(+), 17 deletions(-)

Approvals:
  Martin Peřina: Looks good to me, approved
  Juan Hernandez: Looks good to me, but someone else must approve
  Ravi Nori: Verified
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/78533
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I049acc6ab02e173ae52aa12133c0a13eb1ec223b
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list