DO NOT REPLY [Bug 29914] New: - StandardServer.storeConfig() not saved DefaultContext Listeners

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=29914>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=29914

StandardServer.storeConfig() not saved DefaultContext Listeners

           Summary: StandardServer.storeConfig() not saved DefaultContext
                    Listeners
           Product: Tomcat 5
           Version: 5.0.25
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Catalina
        AssignedTo: tomcat-dev@jakarta.apache.org
        ReportedBy: pr@objektpark.de


Hey,

storeDefaultContext()
===========================================
Hey,

I implement a new store server.xml mechanisnm and detectet following strange
behaviour: 

Config 

<DefaultContext>
   <Listener className="org.apache.catalina.listener.InfoLifecycleListener"/>
</DefaultContext>

are not saved with StandardServer.storeConfig()

1) wrong Lifecycle Listener storage
	StandardDefaultContext not implement Lifecycle.
	Reason: the configuered LifecycleListener not saved at DefaultContext instead
	all Listener saved at every Context. Hups!
	
	s. StandardDefaultContext.installDefaultContext addLifecycleListener
	
	This store mechaismn generate an error situation after deploy new applications

Hmm at my implementation I made following changes:
- delete the Lifecycle instanceof check 
- check every Listener at context
there are not at Engine or Host DefaultContext!
- Also check
2) ResourceLink not saved at storeNamingResource
	correct for GlobalNamingResource but wrong for DefaultContext and Context

3)  After storeConfig the DefaultContext looks like this
       <DefaultContext
className="org.apache.catalina.core.StandardDefaultContext" reloadable="true">

     strange effect this class is included at the skippables[] array!

regards
Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org