Change in ovirt-engine[ovirt-engine-3.6]: webadmin: encoding issue in vv file

tnisan at redhat.com tnisan at redhat.com
Tue Sep 8 14:00:04 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: webadmin: encoding issue in vv file
......................................................................


webadmin: encoding issue in vv file

If some part of the .vv file contains some national chars, it will be
incorrectly encoded.

Fixed by sending them as URL encoded and decode them on server.

Change-Id: I178249bbd779787b02545e40c1c0d0d4137a75df
Bug-Url: https://bugzilla.redhat.com/1258759
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M backend/manager/modules/services/src/main/java/org/ovirt/engine/core/services/GetAttachmentServlet.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ConsoleModel.java
2 files changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Tomas Jelinek: Verified
  Tal Nisan: Passed CI tests
  Vojtech Szocs: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I178249bbd779787b02545e40c1c0d0d4137a75df
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list