usabuts / run-jetty-run

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

Make servlet3 web-fragment.xml working properly in Jetty8 #103

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

MetaInfConfiguration only scan the jars in the WEB-INF/lib ,

if the jar is coming from Maven or other project , it's not working.

It's very similar with Issue 85.

Original issue reported on code.google.com by tonylovejava on 13 Dec 2011 at 3:25

GoogleCodeExporter commented 9 years ago
I found the META-INF/reosurces folder and tld files is also not working for 
class folders.

Original comment by tonylovejava on 14 Dec 2011 at 4:55

GoogleCodeExporter commented 9 years ago
I believe Jetty 7 should have same issue , since they use same configuration 
classes.

Need to review this later.

Original comment by tonylovejava on 14 Dec 2011 at 4:57

GoogleCodeExporter commented 9 years ago

Original comment by tonylovejava on 14 Dec 2011 at 4:58

GoogleCodeExporter commented 9 years ago
It's ready since 2011/12/14 , will be 1.3.2 

Referenced commit log for r270 , r 271 .

Since it's actually the jetty issue , so I want to explain more about this.

I check the Jetty implementation of WebInfConfiguration.

In old version (Jetty6 and Jetty7 ), it's not doing the scan job , 
just simply add those items into the classloader.

And we also add our classpath into the classloader in our bootstrap , so it's 
no issues.

---------

For Jetty 8 with servlet3 implementation , because it need to do some resource 
manangement , 
like serve META-INF/resources as static resources, and new spec for 
META-INF/*.tld .

It just assumed to scan the jars under WEB-INF/lib, so it's not working for 
those custom classpath / workspace projects.

I overwrite some Configuration Class and try to make a better development 
support for those feature.

Original comment by tonylovejava on 14 Dec 2011 at 5:12

GoogleCodeExporter commented 9 years ago

Original comment by tonylovejava on 14 Dec 2011 at 5:13

GoogleCodeExporter commented 9 years ago

Original comment by tonylovejava on 19 Dec 2011 at 6:34