Return-Path: Delivered-To: apmail-jakarta-log4j-dev-archive@apache.org Received: (qmail 48535 invoked from network); 4 Feb 2003 19:04:49 -0000 Received: from exchange.sun.com (192.18.33.10) by daedalus.apache.org with SMTP; 4 Feb 2003 19:04:49 -0000 Received: (qmail 10352 invoked by uid 97); 4 Feb 2003 19:06:19 -0000 Delivered-To: qmlist-jakarta-archive-log4j-dev@nagoya.betaversion.org Received: (qmail 10345 invoked from network); 4 Feb 2003 19:06:19 -0000 Received: from daedalus.apache.org (HELO apache.org) (208.185.179.12) by nagoya.betaversion.org with SMTP; 4 Feb 2003 19:06:19 -0000 Received: (qmail 48186 invoked by uid 500); 4 Feb 2003 19:04:42 -0000 Mailing-List: contact log4j-dev-help@jakarta.apache.org; run by ezmlm Precedence: bulk List-Unsubscribe: List-Subscribe: List-Help: List-Post: List-Id: "Log4J Developers List" Reply-To: "Log4J Developers List" Delivered-To: mailing list log4j-dev@jakarta.apache.org Received: (qmail 48092 invoked from network); 4 Feb 2003 19:04:40 -0000 Received: from ns1.syntegra.com (150.143.16.2) by daedalus.apache.org with SMTP; 4 Feb 2003 19:04:40 -0000 Received: from [129.179.161.11] by ns1.syntegra.com with ESMTP for log4j-dev@jakarta.apache.org; Tue, 4 Feb 2003 19:04:43 Z Received: from ns1.syntegra.com by cdsms.cdc.com with ESMTP for log4j-dev@jakarta.apache.org; Tue, 4 Feb 2003 13:04:42 -0600 Received: from cis-exchange.jedi.local by ns1.syntegra.com with ESMTP for log4j-dev@jakarta.apache.org; Tue, 4 Feb 2003 19:04:42 Z Received: from us613005.ah.us.intra.syntegra.com ([129.179.77.171] unverified) by cis-exchange.jedi.local with Microsoft SMTPSVC(5.0.2195.5329); Tue, 4 Feb 2003 13:04:42 -0600 Date: Tue, 4 Feb 2003 13:04:41 -0600 From: Jacob Kjome X-Mailer: The Bat! (v1.54/10) Personal Reply-To: Jacob Kjome Organization: Springfield Nuclear Plant X-Priority: 3 (Normal) Message-Id: <10510348971.20030204130441@visi.com> To: "Log4J Developers List" Subject: Re[2]: log4j-sandbox options In-Reply-To: References: X-OriginalArrivalTime: 04 Feb 2003 19:04:42.0215 (UTC) FILETIME=[46010F70:01C2CC80] MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N Hello Mark, Yep, Hi Mark, This is exactly why I spoke up about this. Even if Ceki remembered the resolution, I don't think the consensus was ever made that clear. So, at a minimum, we need: log4j-selector.jar log4j.jar log4j-sandbox.jar I would go further and separate any stuff that uses the servlet API into its own jar. Servlet stuff should exist in the WebappClassLoader and I imagine that there is some sandbox stuff besides the selectors that we might want to put alongside the log4j.jar in a shared class loader. Since servlet stuff shouldn't be shared (and absolutely can't be shared in the case of servlets/listeners using the ContextClassLoaderSelector), they should be available in their own jar to be added to WEB-INF/lib of each webapp independently of anything else in the sandbox. So, I propose the following jar: log4j-servlet.jar or log4j-webapp.jar How does that sound? Jake Tuesday, February 04, 2003, 12:31:46 PM, you wrote: MW> But, does this mean that the selectors should be packaged in their own jar, MW> separate from the log4j-sandbox jar? MW> -Mark >> -----Original Message----- >> From: Jacob Kjome [mailto:hoju@visi.com] >> Sent: Tuesday, February 04, 2003 6:40 AM >> To: Log4J Developers List >> Subject: Re: log4j-sandbox options >> >> >> >> Hi Ceki, >> >> That sounds fine. I must have forgotten about this resolution. >> >> Sorry for the bother. >> >> Jake >> >> At 03:34 PM 2/4/2003 +0100, you wrote: >> >> >Hi Jake, >> > >> >I thought we had this covered... It would fine to have >> >log4j.jar+selector.jar available to the shared class loader. >> This way >> >log4j+selectors would be accessible to Servlet Container >> class laoder and >> >the web-application class loaders. This results in one copy of >> >log4j.jar+selector.jar but multiple logger repositories. There is no >> >absolute need to have selector.jar merged into log4j.jar. >> > >> >At 08:28 04.02.2003 -0600, you wrote: >> >>One comment I have is that the servlet stuff needs to be run under >> >>WEB-INF/lib of a webapp whereas the selectors need to be >> wherever an >> >>existing log4j.jar is because there is two-way >> communication between the >> >>selector and log4j proper. This is especially true for the >> >>ContextClassLoaderSelector. The class calling this >> selector must be in >> >>the WebappClassLoader so that the selector gets a handle on >> the class >> >>loader to use as a key for the app's logger repository. >> >> >> >>The problem is that if we package the selectors and the >> servlets/servlet >> >>context listeners in the same jar file, things will break >> unless you put >> >>the sandbox jar and the log4j jar in WEB-INF/lib which >> totally defeats >> >>the purpose of using the custom selector. With everything in >> >>WEB-INF/lib, log4j already has a unique logging environment >> and has no >> >>need to use a custom selector. >> > >> >-- >> >Ceki >> > >> >--------------------------------------------------------------------- >> >To unsubscribe, e-mail: log4j-dev-unsubscribe@jakarta.apache.org >> >For additional commands, e-mail: log4j-dev-help@jakarta.apache.org >> MW> --------------------------------------------------------------------- MW> To unsubscribe, e-mail: log4j-dev-unsubscribe@jakarta.apache.org MW> For additional commands, e-mail: log4j-dev-help@jakarta.apache.org -- Best regards, Jacob mailto:hoju@visi.com --------------------------------------------------------------------- To unsubscribe, e-mail: log4j-dev-unsubscribe@jakarta.apache.org For additional commands, e-mail: log4j-dev-help@jakarta.apache.org