Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin: Check HTTP response status for Reports.xml ...................................................................... webadmin: Check HTTP response status for Reports.xml In ReportInit.parseReportsXML method, HTTP response status should be checked first, before trying to parse response text as XML. For example, missing Reports.xml will cause 404 (not found) HTTP response, with response text being HTML (error page). Change-Id: I40823031291c6223cf9acb484e5b20d74baa8142 Signed-off-by: Vojtech Szocs <vszocs@redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/ReportInit.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7512 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40823031291c6223cf9acb484e5b20d74baa8142 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan@redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server