Skip to content

Comments

Synchronize webdefault.xml and web.xml with Jetty 12#4153

Open
dsmiley wants to merge 1 commit intoapache:mainfrom
dsmiley:webxmlee10
Open

Synchronize webdefault.xml and web.xml with Jetty 12#4153
dsmiley wants to merge 1 commit intoapache:mainfrom
dsmiley:webxmlee10

Conversation

@dsmiley
Copy link
Contributor

@dsmiley dsmiley commented Feb 21, 2026

I noticed our web.xml and webdefault.xml have become out of date, especially since they were not referring to jakarta.

For debdefault.xml, I copied the latest version and made some edits (see inline review). I reviewed the changelog of this file to see where we made intentional edits.

Not sure if needs a JIRA... not really aiming for a side-effect. Albeit it's nice to see a useless log message go away about JSP :-) No more:

2026-02-21 03:49:10.978 INFO (main) [] o.e.j.e.w.StandardDescriptorProcessor NO JSP Support for /solr, did not find org.apache.jasper.servlet.JspServlet

Copy link
Contributor Author

@dsmiley dsmiley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, I retained the default directory listing as false, even though OOTB Jetty it's true.

xmlns="http://java.sun.com/xml/ns/javaee"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/web-app_2_5.xsd"
metadata-complete="true"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the default is actually metadata-complete="false", albeit it doesn't matter since we are explicit in our web.xml.

</servlet-mapping>


<!-- ==================================================================== -->
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed this, even though it's in the base/template.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant