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(a)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(a)redhat.com>
Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org