You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2019/07/24 15:29:18 UTC
[tomcat] branch 8.5.x updated: Fix
https://bz.apache.org/bugzilla/show_bug.cgi?id=63597
This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 8.5.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/8.5.x by this push:
new 942d4d7 Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=63597
942d4d7 is described below
commit 942d4d76784d902d71de94be9d8f4f2f991031de
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Wed Jul 24 16:26:38 2019 +0100
Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=63597
Update to take account of HTML escaping refactoring
---
webapps/docs/changelog.xml | 9 +++++++++
webapps/host-manager/WEB-INF/jsp/404.jsp | 4 ++--
2 files changed, 11 insertions(+), 2 deletions(-)
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 8416e8a..479390d 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -99,6 +99,15 @@
</fix>
</changelog>
</subsection>
+ <subsection name="Web applications">
+ <changelog>
+ <fix>
+ <bug>63597</bug>: Update the custom 404 error page for the Host Manager
+ to take account of previous refactoring so that the page is used for
+ 404 errors rather than falling back to the default error page. (markt)
+ </fix>
+ </changelog>
+ </subsection>
<subsection name="Other">
<changelog>
<update>
diff --git a/webapps/host-manager/WEB-INF/jsp/404.jsp b/webapps/host-manager/WEB-INF/jsp/404.jsp
index 9816df5..1950b77 100644
--- a/webapps/host-manager/WEB-INF/jsp/404.jsp
+++ b/webapps/host-manager/WEB-INF/jsp/404.jsp
@@ -14,7 +14,7 @@
See the License for the specific language governing permissions and
limitations under the License.
--%>
-<%@ page import="org.apache.catalina.util.RequestUtil" session="false"
+<%@ page import="org.apache.tomcat.util.security.Escape" session="false"
trimDirectiveWhitespaces="true" %>
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
<html>
@@ -33,7 +33,7 @@
<h1>404 Not found</h1>
<p>
The page you tried to access
- (<%=RequestUtil.filter((String) request.getAttribute(
+ (<%=Escape.htmlElementContent((String) request.getAttribute(
"javax.servlet.error.request_uri"))%>)
does not exist.
</p>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org