Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 352575 - config/org.eclipse.virgo.web.properties is superseded by repository/ext/org.eclipse.virgo.web.properties
Summary: config/org.eclipse.virgo.web.properties is superseded by repository/ext/org.e...
Status: CLOSED FIXED
Alias: None
Product: Virgo
Classification: RT
Component: runtime (show other bugs)
Version: 3.0.0.M06   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: 3.0.0.RC1   Edit
Assignee: Glyn Normington CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-20 09:22 EDT by Glyn Normington CLA
Modified: 2011-07-21 11:07 EDT (History)
0 users

See Also:
glyn.normington: documentation+

Attachments
WAB without Bundle-ClassPath (3.54 KB, application/octet-stream)
2011-07-20 09:22 EDT, Glyn Normington CLA 		no flags Details
WAB with Bundle-Classpath (3.50 KB, application/java-archive)
2011-07-20 09:22 EDT, Glyn Normington CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.