Change in ovirt-engine[ovirt-engine-3.4]: engine: docs internal server error
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(a)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(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server