It looks like pax web 0.8.1 correctly serves content without it needing to be on the classpath.
I propose we move to 0.8.1 rather than 0.7.2 and don't add . onto the classpath. What do you think? Alasdair On 11 January 2011 22:22, Alasdair Nottingham <[email protected]> wrote: > You are right about the spec and WEB-INF, but they didn't implement the WAB > spec properly so who knows, but it looks like we should be safe. > > Alasdair Nottingham > > On 11 Jan 2011, at 22:09, Joe Bohn <[email protected]> wrote: > >> Yes, that is correct. I found that I had to do this to get the static >> content served. After your changes to create of the web bundles the >> bundle-classpath was added to the manifest and static content was no longer >> served. I didn't think this would serve up the class files as I thought >> that a spec compliant implementation would not serve any static content from >> WEB-INF/. >> >> Joe >> >> On 1/11/11 4:31 PM, Alasdair Nottingham wrote: >>> Hi, >>> >>> This is a really bad change to make. It essentially means that the >>> class files will be served up. The web extender should not require the >>> content to be available on the classpath. Did you make this change to >>> get the static content served? If so I'll raise another bug against >>> pax-web. >>> >>> On 11 January 2011 15:51,<[email protected]> wrote: >>>> Author: jbohn >>>> Date: Tue Jan 11 15:51:16 2011 >>>> New Revision: 1057705 >>>> >>>> URL: http://svn.apache.org/viewvc?rev=1057705&view=rev >>>> Log: >>>> merge 1057701 from trunk >>>> >>>> Modified: >>>> >>>> aries/branches/0.3-RCx/samples/ariestrader/modules/ariestrader-web/pom.xml >>>> aries/branches/0.3-RCx/samples/blog/blog-web/pom.xml >>>> >>>> Modified: >>>> aries/branches/0.3-RCx/samples/ariestrader/modules/ariestrader-web/pom.xml >>>> URL: >>>> http://svn.apache.org/viewvc/aries/branches/0.3-RCx/samples/ariestrader/modules/ariestrader-web/pom.xml?rev=1057705&r1=1057704&r2=1057705&view=diff >>>> ============================================================================== >>>> --- >>>> aries/branches/0.3-RCx/samples/ariestrader/modules/ariestrader-web/pom.xml >>>> (original) >>>> +++ >>>> aries/branches/0.3-RCx/samples/ariestrader/modules/ariestrader-web/pom.xml >>>> Tue Jan 11 15:51:16 2011 >>>> @@ -70,6 +70,7 @@ >>>> <instructions> >>>> >>>> <Bundle-SymbolicName>${pom.artifactId}</Bundle-SymbolicName> >>>> <Bundle-Version>${pom.version}</Bundle-Version> >>>> +<Bundle-ClassPath>.</Bundle-ClassPath> >>>> <Web-ContextPath>/ariestrader</Web-ContextPath> >>>> <Web-JSPExtractLocation>/</Web-JSPExtractLocation> >>>> >>>> <Private-Package>org.apache.aries.samples.ariestrader.web.*</Private-Package> >>>> >>>> Modified: aries/branches/0.3-RCx/samples/blog/blog-web/pom.xml >>>> URL: >>>> http://svn.apache.org/viewvc/aries/branches/0.3-RCx/samples/blog/blog-web/pom.xml?rev=1057705&r1=1057704&r2=1057705&view=diff >>>> ============================================================================== >>>> --- aries/branches/0.3-RCx/samples/blog/blog-web/pom.xml (original) >>>> +++ aries/branches/0.3-RCx/samples/blog/blog-web/pom.xml Tue Jan 11 >>>> 15:51:16 2011 >>>> @@ -50,6 +50,7 @@ >>>> <instructions> >>>> <Web-ContextPath>/blog</Web-ContextPath> >>>> >>>> <Bundle-SymbolicName>${pom.artifactId}</Bundle-SymbolicName> >>>> +<Bundle-ClassPath>.</Bundle-ClassPath> >>>> >>>> <Private-Package>org.apache.aries.samples.blog.web.*</Private-Package> >>>> >>>> <Export-Package>!org.apache.aries.samples.blog.web.*</Export-Package> >>>> <Import-Package> >>>> >>>> >>>> >>> >>> >>> >> >> >> -- >> Joe > -- Alasdair Nottingham [email protected]
