Change in ovirt-engine[ovirt-engine-3.4]: engine: docs internal server error

oourfali at redhat.com oourfali at redhat.com
Sun Apr 6 06:52:53 UTC 2014


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 at 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 at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen at redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knesenko at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list