Change in ovirt-engine[master]: userportal: doesn't try SSO

derez at redhat.com derez at redhat.com
Sat Jul 21 20:08:14 UTC 2012


Daniel Erez has submitted this change and it was merged.

Change subject: userportal: doesn't try SSO
......................................................................


userportal: doesn't try SSO

The UserPortal did not try to perform SSO.

The problem was, that the SSO is performed only
when the Configurator.privateIsAdmin field is false
(see SpiceConsoleModel.PostSendVmTicket), but it was
always true.

Fixed by setting this field separatly to true for WebAdmin
(WebAdminConfigurator) and to false (UserPortalConfigurator).

Change-Id: I470e21f5e23a603b1f2cfc415e73e99b9c899e08
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Configurator.java
M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/uicommon/UserPortalConfigurator.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/uicommon/WebAdminConfigurator.java
3 files changed, 8 insertions(+), 6 deletions(-)

Approvals:
  Daniel Erez: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/6218
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I470e21f5e23a603b1f2cfc415e73e99b9c899e08
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>



More information about the Engine-commits mailing list