wuh / run-jetty-run

Automatically exported from code.google.com/p/run-jetty-run
0 stars 0 forks source link

Support for ServletContainerInitializer #125

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Servlet 3.0 supports running webapps without web.xml by using 
ServletContainerInitializer. There seems to support for that already in Jetty 
8.0 in a form of ContainerInitializerConfiguration. RJR however doesn't seem to 
include this configuration class as a part of its configuration. Would it be 
possible to enable it by default if using Jetty 8.x version?

Maybe that can be done manually somewhere, but at least I'm not aware of it.

Original issue reported on code.google.com by kjhelen...@gmail.com on 7 Mar 2012 at 2:46

GoogleCodeExporter commented 9 years ago
...and after submitting the issue I got it working by adding 
"-Drjrconfigurationclasses=org.eclipse.jetty.annotations.ContainerInitializerCon
figuration" to VM arguments.

I found it by going through the source code. Maybe it should be mentioned in 
the documentation?

Original comment by kjhelen...@gmail.com on 7 Mar 2012 at 3:07

GoogleCodeExporter commented 9 years ago
You are right , there's still a lot of un-documents issues ,
I will leave this issue until I document it....;) 

Original comment by tonylovejava on 8 Mar 2012 at 1:33

GoogleCodeExporter commented 9 years ago
After checking more details , I decide to add this configuration by default,
it will be in next release.

Original comment by tonylovejava on 11 Mar 2012 at 3:37

GoogleCodeExporter commented 9 years ago

Original comment by tonylovejava on 11 Mar 2012 at 4:54

GoogleCodeExporter commented 9 years ago
After we upgrade to JEtty 8.1.2 v20120308 , 
the  ContainerInitializerConfiguration  class is gone and there's no any 
description in release note. 

So we remove the configuration first, 
user could still add their own configuration by the system properties when need.

Original comment by tonylovejava on 16 Mar 2012 at 3:19

GoogleCodeExporter commented 9 years ago

Original comment by tonylovejava on 16 Mar 2012 at 3:23