tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jan Luehe <Jan.Lu...@Sun.COM>
Subject Re: cvs commit: jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/valves
Date Thu, 03 Mar 2005 02:22:11 GMT

Remy Maucherat wrote:
> Jan Luehe wrote:
>>Remy Maucherat wrote:
>>>Actually, this is probably a bad idea (or at least the implementation is 
>>>bad): the logger must be retrieved only when the context class loader of 
>>>the application is set.
>>This is where I'm not following: ;-)
>>I don't see any dependency on the context's context class loader when
>>calling ContainerBase.getLogger(), which is implemented like this:
>>  logger = LogFactory.getLog(logName());
>>A context's context class loader is required only for loading webapp
> The configuration for that logger should be doable at the webapp level 
> (to summarize, I'd like to be able to have my logger configured using 
> either /WEB-INF/classes/ or 
> common/classes/ if the first one does not exist). The 
> logging implementation gets which config it should lookup for the logger 
> based on what the context CL is when calling getLogger. So the context 
> CL needs to be set properly (or getLogger has to be called again).

Got it. Thanks!


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message