This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new 0746ea9be8 Improve HTML output of <code>DefaultServlet
0746ea9be8 is described below

commit 0746ea9be8dea5d43b82c5c0168b6525da22068f
Author: Michael Osipov <micha...@apache.org>
AuthorDate: Sat Oct 5 19:18:17 2024 +0200

    Improve HTML output of <code>DefaultServlet
---
 java/org/apache/catalina/servlets/DefaultServlet.java | 17 +++++++++--------
 webapps/docs/changelog.xml                            |  3 +++
 2 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/java/org/apache/catalina/servlets/DefaultServlet.java 
b/java/org/apache/catalina/servlets/DefaultServlet.java
index 4ec1a3bdd0..3b33c0fc29 100644
--- a/java/org/apache/catalina/servlets/DefaultServlet.java
+++ b/java/org/apache/catalina/servlets/DefaultServlet.java
@@ -1744,7 +1744,8 @@ public class DefaultServlet extends HttpServlet {
         String rewrittenContextPath = rewriteUrl(contextPath);
 
         // Render the page header
-        sb.append("<!doctype html><html>\r\n");
+        sb.append("<!doctype html>\r\n");
+        sb.append("<html>\r\n");
         /*
          * TODO Activate this as soon as we use smClient with the request 
locales
          * sb.append("<!doctype html><html lang=\""); 
sb.append(smClient.getLocale().getLanguage()).append("\">\r\n");
@@ -1755,9 +1756,9 @@ public class DefaultServlet extends HttpServlet {
         sb.append("</title>\r\n");
         sb.append("<style>");
         sb.append(org.apache.catalina.util.TomcatCSS.TOMCAT_CSS);
-        sb.append("</style> ");
+        sb.append("</style>\r\n");
         sb.append("</head>\r\n");
-        sb.append("<body>");
+        sb.append("<body>\r\n");
         sb.append("<h1>");
         sb.append(sm.getString("directory.title", directoryWebappPath));
 
@@ -1785,8 +1786,8 @@ public class DefaultServlet extends HttpServlet {
             sb.append("</a>");
         }
 
-        sb.append("</h1>");
-        sb.append("<hr class=\"line\">");
+        sb.append("</h1>\r\n");
+        sb.append("<hr class=\"line\">\r\n");
 
         sb.append("<table width=\"100%\" cellspacing=\"0\"" + " 
cellpadding=\"5\" align=\"center\">\r\n");
 
@@ -1891,16 +1892,16 @@ public class DefaultServlet extends HttpServlet {
         // Render the page footer
         sb.append("</table>\r\n");
 
-        sb.append("<hr class=\"line\">");
+        sb.append("<hr class=\"line\">\r\n");
 
         String readme = getReadme(resource, encoding);
         if (readme != null) {
             sb.append(readme);
-            sb.append("<hr class=\"line\">");
+            sb.append("<hr class=\"line\">\r\n");
         }
 
         if (showServerInfo) {
-            
sb.append("<h3>").append(ServerInfo.getServerInfo()).append("</h3>");
+            
sb.append("<h3>").append(ServerInfo.getServerInfo()).append("</h3>\r\n");
         }
         sb.append("</body>\r\n");
         sb.append("</html>\r\n");
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 54f8f7b756..98424af9e3 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -115,6 +115,9 @@
         <bug>69373</bug>: Make <code>DefaultServlet</code>'s HTML listing
         file last modified rendering better (flexible). (michaelo)
       </update>
+      <update>
+        Improve HTML output of <code>DefaultServlet</code>. (michaelo)
+      </update>
     </changelog>
   </subsection>
 </section>


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to