Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 352575

Summary: config/org.eclipse.virgo.web.properties is superseded by repository/ext/org.eclipse.virgo.web.properties
Product: [RT] Virgo Reporter: Glyn Normington <glyn.normington>
Component: runtimeAssignee: Glyn Normington <glyn.normington>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 Flags: glyn.normington: documentation+
Version: 3.0.0.M06   
Target Milestone: 3.0.0.RC1   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:
Attachments:
Description Flags
WAB without Bundle-ClassPath
none
WAB with Bundle-Classpath none

Description Glyn Normington CLA 2011-07-20 09:22:19 EDT 
Created attachment 199993 [details]
WAB without Bundle-ClassPath

The attached file session-persist-test.war (from bug 350154) fails when a request is sent (at http://localhost:8080/sess/Test) due to a class loading error for a class which should be on the bundle classpath. The file is actually a Web Application Bundle rather than a standard WAR since it sets Web-ContextPath in the manifest. As of 3.0.0.M06, Bundle-Classpath is not set to a default value for WABs unless WABHeaders is set to defaulted in the web integration layer properties file.

The bug here is that setting WABHeaders to defaulted still does not fix the class loading problem. Explicitly setting the bundle classpath, as in the attached file session-persist-test.jar solves the problem. So it appears the defaulting is not happening as expected when WABHeaders is set to defaulted.
Comment 1 Glyn Normington CLA 2011-07-20 09:22:46 EDT 
Created attachment 199994 [details]
WAB with Bundle-Classpath
Comment 2 Glyn Normington CLA 2011-07-20 09:36:46 EDT 
The problem is that the org.eclipse.virgo.web.properties file in config is loaded into the kernel instance of Config Admin, but the web layer runs in the user region. The solution is to move the properties file into repository/ext. The documentation needs to be reworked to match.
Comment 3 Glyn Normington CLA 2011-07-21 05:41:07 EDT 
Fixed, but waiting to push and ripple.

Web integration layer fix in commits 9f42e155268597824b0eedcf7502c68a8b7aa475 and 82f2dc4eab22c368609bd85ea78084de0a44d057.

Documentation fix in commit fc93043af3ee1663b04744856702a5ec55353268.

Virgo Tomcat Server fix in commit d6e3c059e8be4ee6b62427d8fb40863cba44fff7.
Comment 4 Glyn Normington CLA 2011-07-21 11:07:04 EDT 
Rippled and pushed.