Nice.
On Thu, Sep 4, 2014 at 8:32 PM, Matt Sicker boa...@gmail.com wrote:
I've fixed this in the master branch.
On 4 September 2014 11:24, Mikael Ståldal
mikael.stal...@appearnetworks.com
wrote:
I think it makes sense to log to ServletContext.log() if something goes
wrong, but not
FWIW, I do not see any logging in Log4jServletContextListener ;-)
G
On Thu, Sep 4, 2014 at 12:03 PM, Mikael Ståldal
mikael.stal...@appearnetworks.com wrote:
Log4jServletContainerInitializer, Log4jServletContextListener,
Log4jServletFilter and Log4jWebInitializerImpl log some stuff using
Should be log only if some condition is met? Like the status logger level
set to DEBUG?
Gary
On Thu, Sep 4, 2014 at 12:03 PM, Mikael Ståldal
mikael.stal...@appearnetworks.com wrote:
Log4jServletContainerInitializer, Log4jServletContextListener,
Log4jServletFilter and Log4jWebInitializerImpl
Perhaps we should log to StatusLogger rather than to ServletContext.log()?
I mean, we even have an appender that goes to the ServletContext log, so it
might be a better idea to just use the StatusLogger as usual (unless an
error occurs?).
On 4 September 2014 11:07, Gary Gregory
I see this line in the source code for version 2.0.2 (line 36):
this.servletContext.log(Log4jServletContextListener ensuring that
Log4j starts up properly.);
On Thu, Sep 4, 2014 at 6:06 PM, Gary Gregory garydgreg...@gmail.com wrote:
FWIW, I do not see any logging in
I think it makes sense to log to ServletContext.log() if something goes
wrong, but not otherwise.
Shouldn't we consider ServletContext.log() as deprecated, and Log4j as the
replacement for it?
On Thu, Sep 4, 2014 at 6:07 PM, Gary Gregory garydgreg...@gmail.com wrote:
Should be log only if
I've fixed this in the master branch.
On 4 September 2014 11:24, Mikael Ståldal mikael.stal...@appearnetworks.com
wrote:
I think it makes sense to log to ServletContext.log() if something goes
wrong, but not otherwise.
Shouldn't we consider ServletContext.log() as deprecated, and Log4j as