Re: [Webwork-devel] loading log4j.properties

[Matt B. <ma...@sm...>]

For now I made the proposed change but I agree with your point. I need to
finish the JSP doc and I will look at this next.

-Matt

----- Original Message -----
From: "Mike Cannon-Brookes" <boo...@ya...>
To: "Matt Baldree" <ma...@sm...>; "Webwork-Developer"
<web...@li...>
Sent: Monday, January 07, 2002 10:16 PM
Subject: Re: [Webwork-devel] loading log4j.properties


> Matt,
>
> I brought this up a while ago because I have a
> different issue. I always use xml configuration files
> in log4j (easier to manipulate, generate etc) and
> there's no way to do that.
>
> What I'd suggest is abstract the log loading into
> separate class that's easily extendable. The default
> behaviour as below is good, and also if log4j is
> already configured it should not touch it at all. (I
> configure logging at the EAR level).
>
> -mike
>
> --- Matt Baldree <ma...@sm...> wrote:
> > I was working on the JSP documentation and realized
> > we need a configuration
> > document to describe how to configure WW. I wrote
> > this today and added it to
> > CVS. One section in configuration talks about
> > configuring WW's logger. I
> > think the developer should be able to substitute the
> > default
> > log4j.properties under webwork/ with their own if
> > they so choose. This means
> > they could provide a log4j.properties under
> > WEB-INF/classes that WW would
> > first check before loading its default under
> > webwork/. This allows the
> > developer a clean log4j.properties substitution
> > without having to
> > programmatically do it somewhere else. To allow this
> > capability, I propose
> > changing the ServletDispatcher as follows:
> >
> > 84c84,92
> > <
> >
>
PropertyConfigurator.configure(classLoader.getResource("webwork/log4j.proper
> > ties"));
> > ---
> > >       try
> > >       {
> > >
> >
>
PropertyConfigurator.configure(classLoader.getResource("log4j.properties"));
> > >          log.info("Custom log4j property file
> > loaded.");
> > >       } catch (Exception e)
> > >       {
> > >
> >
>
PropertyConfigurator.configure(classLoader.getResource("webwork/log4j.proper
> > ties"));
> > >          log.info("Default log4j property file
> > loaded.");
> > >       }
> >
> > Thoughts?
> >
> > -Matt
> >
> >
> >
> > _______________________________________________
> > Webwork-devel mailing list
> > Web...@li...
> >
> https://lists.sourceforge.net/lists/listinfo/webwork-devel
>
>
> __________________________________________________
> Do You Yahoo!?
> Send FREE video emails in Yahoo! Mail!
> http://promo.yahoo.com/videomail/
>
>