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

tjelinek at redhat.com tjelinek at redhat.com
Mon Sep 7 14:37:07 UTC 2015


Tomas Jelinek 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
  Jenkins CI: Passed CI tests
  Martin Betak: Looks good to me, but someone else must approve
  Vojtech Szocs: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I178249bbd779787b02545e40c1c0d0d4137a75df
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Jakub Niedermertl <jniederm at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek 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