svn commit: r369745 - /jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/LogFactory.java

[sk...@apache.org]

Author: skitching
Date: Tue Jan 17 02:39:30 2006
New Revision: 369745

URL: http://svn.apache.org/viewcvs?rev=369745&view=rev
Log:
If config file has use_tccl=false, then don't load LogFactory implementation from the TCCL.

Modified:
    jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/LogFactory.java

Modified: jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/LogFactory.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/LogFactory.java?rev=369745&r1=369744&r2=369745&view=diff
==============================================================================
--- jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/LogFactory.java (original)
+++ jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/LogFactory.java Tue Jan 17 02:39:30 2006
@@ -61,6 +61,13 @@
     public static final String PRIORITY_KEY = "priority";
 
     /**
+     * The name of the key in the config file used to specify whether logging
+     * classes should be loaded via the thread context class loader (TCCL),
+     * or not. By default, the TCCL is used.
+     */
+    public static final String TCCL_KEY = "use_tccl";
+
+    /**
      * The name of the property used to identify the LogFactory implementation
      * class name. This can be used as a system property, or as an entry in a
      * configuration properties file.
@@ -391,6 +398,25 @@
 
         Properties props = getConfigurationFile(contextClassLoader, FACTORY_PROPERTIES);
 
+        // Determine whether we will be using the thread context class loader to
+        // load logging classes or not by checking the loaded properties file (if any).
+        ClassLoader baseClassLoader = contextClassLoader;
+        if (props != null) {
+            String useTCCLStr = props.getProperty(TCCL_KEY);
+            if (useTCCLStr != null) {
+                if (Boolean.valueOf(useTCCLStr) == Boolean.FALSE) {
+                    // Don't use current context classloader when locating any
+                    // LogFactory or Log classes, just use the class that loaded
+                    // this abstract class. When this class is deployed in a shared
+                    // classpath of a container, it means webapps cannot deploy their
+                    // own logging implementations. It also means that it is up to the
+                    // implementation whether to load library-specific config files
+                    // from the TCCL or not.
+                    baseClassLoader = thisClassLoader; 
+                }
+            }
+        }
+
         // Determine which concrete LogFactory subclass to use.
         // First, try a global system property
         logDiagnostic(
@@ -404,7 +430,7 @@
                     "Creating an instance of LogFactory class " + factoryClass
                     + " as specified by system property " + FACTORY_PROPERTY);
 
-                factory = newFactory(factoryClass, contextClassLoader, contextClassLoader);
+                factory = newFactory(factoryClass, baseClassLoader, contextClassLoader);
             }
         } catch (SecurityException e) {
             logDiagnostic(
@@ -453,7 +479,7 @@
                             + " which was present in the path of the context"
                             + " classloader.");
 
-                        factory = newFactory( factoryClassName, contextClassLoader, contextClassLoader );
+                        factory = newFactory(factoryClassName, baseClassLoader, contextClassLoader );
                     }
                 }
             } catch( Exception ex ) {
@@ -489,7 +515,7 @@
 
                 String factoryClass = props.getProperty(FACTORY_PROPERTY);
                 if (factoryClass != null) {
-                    factory = newFactory(factoryClass, contextClassLoader, contextClassLoader);
+                    factory = newFactory(factoryClass, baseClassLoader, contextClassLoader);
                     
                     // what about handling an exception from newFactory??
                 }
@@ -1002,6 +1028,7 @@
             logDiagnostic("Unable to create logfactory instance.");
             if (logFactoryClass != null
                 && !LogFactory.class.isAssignableFrom(logFactoryClass)) {
+                
                 return new LogConfigurationException(
                     "The chosen LogFactory implementation does not extend LogFactory."
                     + " Please check your configuration.",



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org