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

Bug 318152

Summary: Ability to locate other files in the web-app inside the jetty-web.xml configuration file
Product: [RT] Jetty Reporter: Hugues Malphettes <hmalphettes>
Component: serverAssignee: Hugues Malphettes <hmalphettes>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: jetty-inbox, mgorovoy
Version: 7.1.4   
Target Milestone: 7.1.x   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Patch to setup the property 'this.web-inf.url' to the jetty-web.xml configuration
none
Same patch minus one trivial bug none

Description Hugues Malphettes CLA 2010-06-28 05:46:47 EDT
Testcase:
the configuration of the test-jetty-webapp uses the HashLoginService which itself refers to a properties file.

The enhancement consists of being able to embed the configuration of the properties file. It requires that when the jetty-web.xml is read by the WebInfConfiguration a new property is set with the URL of the WEB-INF folder.

<Configure class="org.eclipse.jetty.webapp.WebAppContext">
...
  <Get name="securityHandler">
    <Set name="loginService">
      <New class="org.eclipse.jetty.security.HashLoginService">
	    <Set name="name">Test Realm</Set>
	    <Set name="config"><Property name="this.web-inf.url"/>realm.properties</Set>
      </New>
    </Set>
    <Set name="checkWelcomeFiles">true</Set>
  </Get>
</Configure>
Comment 1 Hugues Malphettes CLA 2010-06-28 05:49:15 EDT
Created attachment 172878 [details]
Patch to setup the property 'this.web-inf.url' to the jetty-web.xml configuration
Comment 2 Hugues Malphettes CLA 2010-06-28 06:01:09 EDT
Created attachment 172881 [details]
Same patch minus one trivial bug
Comment 3 Hugues Malphettes CLA 2010-06-28 16:28:52 EDT
Greg the patch has been committed. Pending your review I'll document it on the wiki page related to jetty-web.xml
Comment 4 Greg Wilkins CLA 2010-06-30 19:50:29 EDT
Jan,

can you look at this patch from hugues and see if it fits with your new configuration stuff that is coming?
Comment 5 Jan Bartel CLA 2010-11-15 01:28:29 EST
This patch is fine by me. Greg - does it have any implications for the templated deployment of webapps?

thanks
Jan
Comment 6 Greg Wilkins CLA 2011-02-23 00:09:01 EST
applied some time ago in r2055
Comment 7 Jesse McConnell CLA 2011-09-20 15:51:47 EDT
Resolved -> Closed