https://issues.apache.org/bugzilla/show_bug.cgi?id=46221





--- Comment #4 from Arnaud de Bossoreille <[EMAIL PROTECTED]>  2008-12-03 
06:27:36 PST ---
I am also using the manager via the web interface but with the start/stop links
of the webapp.

I am a little bit confused by the plumbing of tomcat's objects, that is why I
do not always see what is involved. However it seems that your patch in
StandardPipeline.java is better than what I did in StandardContext.java for the
same purpose, because it handles more instances than those created by the
StandardContext.

My test showed that 3 valves are concerned by the releaseContainerLog calls.
But maybe not all of them have a reference on the WebappClassLoader (I did not
check). The valves I see (System.err.println(current);) are:

    - org.apache.catalina.core.StandardWrapperValve[jsp]
    - org.apache.catalina.core.StandardWrapperValve[default]
    - org.apache.catalina.core.StandardContextValve[/logging-leak]

I suppose my patch handled only the last one.

Moreover, you may have more valves than I have. This would explain why my patch
did not work for you (I guess).

I created the patch I last used so that you can review it (I will add it as a
new attachement).


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to