Oved Ourfali has submitted this change and it was merged. Change subject: engine: docs internal server error ...................................................................... engine: docs internal server error - If you try to load a manual document in a locale that doesn't exist, the server would throw a 500 due to a problem in the setup of path of the DocsServlet. This patch fixes that issue. - If you try to load a manual document in a locale that doesn't exist, you will get a 404 instead of a warning message that the language is not available. - Fixed issue in the path of the link that would forward you to the english documentation. - Fixed issue in the cookie path that would store if you had been shown the warning message yet. Change-Id: I3036d05e736a4bc2603423fb7041910a9daa1472 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1083528 Signed-off-by: Alexander Wels <awels@redhat.com> --- M backend/manager/modules/docs/src/main/webapp/WEB-INF/web.xml M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/servlet/DocsServlet.java 2 files changed, 5 insertions(+), 4 deletions(-) Approvals: Alexander Wels: Verified Kiril Nesenko: Looks good to me, but someone else must approve Vojtech Szocs: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26392 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3036d05e736a4bc2603423fb7041910a9daa1472 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Alexander Wels <awels@redhat.com> Gerrit-Reviewer: Alexander Wels <awels@redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen@redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs@redhat.com> Gerrit-Reviewer: automation@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server