Yeah, but it's just a bit nicer, as we can already do that using the
etc/jetty.xml, so I think it's not very high priority improvement.

On Friday, December 24, 2010, Charles Moulliard <[email protected]> wrote:
> The less resources + time consuming approach is to extend ConfigAdmin
> (as you propose) and use the file :
>
> org.ops4j.pax.web.cfg file
>
> and adding parameters that we need to instantiate the constraint,
> constraintMapping and securityHandler of Jetty
>
>
> On Fri, Dec 24, 2010 at 8:26 AM, Guillaume Nodet <[email protected]> wrote:
>> On Friday, December 24, 2010, Charles Moulliard <[email protected]> wrote:
>>> Indeed this will not be easy as pax-web must be changed because it
>>> uses a method of jetty to load the resource file (jetty.xml).
>>
>> Yes, that's clearly not an easy task.
>>
>>
>>>
>>>         public void start() {
>>>                 LOG.debug("Starting " + this);
>>>                 try {
>>>                         //PAXWEB-193 suggested we should open this up for 
>>> external configuration
>>>                         URL jettyResource = 
>>> getClass().getResource("/jetty.xml");
>>>                         File serverConfigurationFile = getServerConfigDir();
>>>                         if (serverConfigurationFile != null) {
>>>                                 if (LOG.isDebugEnabled())
>>>                                         LOG.debug("found server 
>>> configuration file: "+serverConfigurationFile);
>>>                                 if (!serverConfigurationFile.isDirectory() 
>>> &&
>>> serverConfigurationFile.canRead()) {
>>>                                         if (LOG.isDebugEnabled()) {
>>>                                                 LOG.debug("server config 
>>> dir is readable and exists");
>>>                                         }
>>>                                         String fileName = 
>>> serverConfigurationFile.getName();
>>>                                         if 
>>> (fileName.equalsIgnoreCase("jetty.xml"))
>>>                                                 jettyResource = 
>>> serverConfigurationFile.toURI().toURL();
>>>                                 }
>>>                         }
>>>                         if (jettyResource != null) {
>>>                                 ClassLoader loader = 
>>> Thread.currentThread().getContextClassLoader();
>>>                                 try
>>>                                 {
>>>                                         
>>> Thread.currentThread().setContextClassLoader(
>>> getClass().getClassLoader() );
>>>                                         LOG.debug("Configure using resource 
>>> " + jettyResource);
>>>                                         XmlConfiguration configuration = 
>>> new XmlConfiguration(jettyResource);
>>>

-- 
Cheers,
Guillaume Nodet
------------------------
Blog: http://gnodet.blogspot.com/
------------------------
Open Source SOA
http://fusesource.com

Reply via email to