Change in ovirt-engine[master]: userportal, webadmin: disable SPICE proxy has no effect

tjelinek at redhat.com tjelinek at redhat.com
Thu Apr 18 08:38:18 UTC 2013


Tomas Jelinek has submitted this change and it was merged.

Change subject: userportal,webadmin: disable SPICE proxy has no effect
......................................................................


userportal,webadmin: disable SPICE proxy has no effect

Edit console dialog -> enable SPICE proxy -> connect to VM->
edit console dialog again -> disable SPICE proxy -> connect to VM

After this, the SPICE will connect to the VM using the proxy,
because it remembered it from the previous setting.

Fixed by resetting the Proxy property if the proxy is not enabled.

Change-Id: I5931b4242bdba7755e29c3942a1df2eab03f7b5f
Bug-Url: https://bugzilla.redhat.com/924605
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpicePluginImpl.java
1 file changed, 10 insertions(+), 4 deletions(-)

Approvals:
  Tomas Jelinek: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5931b4242bdba7755e29c3942a1df2eab03f7b5f
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: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>



More information about the Engine-commits mailing list