appengine-java-vm-runtime icon indicating copy to clipboard operation
appengine-java-vm-runtime copied to clipboard

Change webdefault.xml to use org.eclipse.jetty.jsp.JettyJspServlet

Open janbartel opened this issue 10 years ago • 2 comments

The webdefault.xml currently uses the standard JspServlet, but should be updated to use the JettyJspServlet. See this jetty bug: https://bugs.eclipse.org/bugs/show_bug.cgi?id=452261

The JettyJspServlet supports correct handling of element, particularly in the presence of elements in web.xml.

AppEngine usually precompiles *.jsp files (which means this issue does not come into play), however, it is possible to turn off pre-compilation in the appengine-web.xml so this change should be made.

janbartel avatar Aug 12 '15 07:08 janbartel

As described in issue #34 , although I will change the webdefault.xml in docker/etc, this is NOT the webdefault.xml that will be used when running with the gcloud maven plugin because of the quickstart mechanism.

janbartel avatar Aug 13 '15 08:08 janbartel

@ludoch I've assigned this issue to you to analyse what to do in the case of the multiple copies of webdefault.xml that can be in play when using the gcloud maven plugin.

janbartel avatar Sep 16 '15 22:09 janbartel