Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 318152 - Ability to locate other files in the web-app inside the jetty-web.xml configuration file
Summary: Ability to locate other files in the web-app inside the jetty-web.xml configu...
Status: CLOSED FIXED
Alias: None
Product: Jetty
Classification: RT
Component: server (show other bugs)
Version: 7.1.4   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 7.1.x   Edit
Assignee: Hugues Malphettes CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-28 05:46 EDT by Hugues Malphettes CLA
Modified: 2011-09-20 15:51 EDT (History)
2 users (show)

See Also:


Attachments
Patch to setup the property 'this.web-inf.url' to the jetty-web.xml configuration (2.03 KB, patch)
2010-06-28 05:49 EDT, Hugues Malphettes CLA
no flags Details | Diff
Same patch minus one trivial bug (2.07 KB, patch)
2010-06-28 06:01 EDT, Hugues Malphettes CLA
no flags Details | Diff

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