Bug 38061 - Problem configuring an errorHandler using a property file
Problem configuring an errorHandler using a property file
Product: Log4j
Classification: Unclassified
Component: Configurator
Other Windows XP
: P1 normal
: ---
Assigned To: log4j-dev
Depends on:
  Show dependency tree
Reported: 2005-12-28 12:05 UTC by Luca Falcon
Modified: 2009-10-07 22:26 UTC (History)
1 user (show)


Note You need to log in before you can comment on or make changes to this bug.
Description Luca Falcon 2005-12-28 12:05:40 UTC
I obtain an error configuring an errorHandler using a property file.
If use a xml file, everything is right.

I noticed tha the problem is in the "org.apache.log4j.config.PropertySetter" in
"convertArg" method.
The "if" sequence don't foreseen to manage the errorHandler parameter

protected Object convertArg(String val, Class type)
	if (val == null)
		return null;

	String v = val.trim();
	if (String.class.isAssignableFrom(type))
		return val;
	} else if (Integer.TYPE.isAssignableFrom(type))
		return new Integer(v);
	} else if (Long.TYPE.isAssignableFrom(type))
		return new Long(v);
	} else if (Boolean.TYPE.isAssignableFrom(type))
		if ("true".equalsIgnoreCase(v))
			return Boolean.TRUE;
		} else if ("false".equalsIgnoreCase(v))
			return Boolean.FALSE;
	} else if (Priority.class.isAssignableFrom(type))
		return OptionConverter.toLevel(v, (Level) Level.DEBUG);
	/* BUG FIX                                                                */
	else if(ErrorHandler.class.isAssignableFrom(type)) 
		// I copied this line from "org.apache.log4j.xml.DOMCOnfigurator" 
		// (method: "parseErrorHandler")
		return OptionConverter.instantiateByClassName(val,
	return null;

But at the moment it is not possible set rootLogger, any other loggers or backup
Is these future enhancement? Can post the code?

Luca Falcon

My environment:

the log4j version is:

this is my log4j.properties:

log4j.rootLogger                                 = INFO, systemOut

log4j.appender.systemOut                          = org.apache.log4j.FileAppender
log4j.appender.systemOut.errorHandler             =
log4j.appender.systemOut.Threshold                = INFO
log4j.appender.systemOut.File                     = ${log_dir}/system.log
log4j.appender.systemOut.layout                   = org.apache.log4j.PatternLayout
log4j.appender.systemOut.layout.ConversionPattern = %d{HH:mm:ss,SSS} %-5p [%t] -

this is the output:
log4j:WARN Failed to set property [errorHandler] to value
log4j: Parsing for [root] with value=[INFO, systemOut].
log4j: Level token is [INFO].
log4j: Category root set to INFO
log4j: Parsing appender named "systemOut".
log4j: Parsing layout options for "systemOut".
log4j: Setting property [conversionPattern] to [%d{HH:mm:ss,SSS} %-5p [%t] - %m%n].
log4j: End of parsing for "systemOut".
log4j: Setting property [file] to [D:/temp/log/system.log].
log4j: Setting property [threshold] to [INFO].
log4j: setFile called: D:/temp/log/system.log, true
log4j: setFile ended
log4j: Parsed "systemOut" options.
log4j: Finished configuring.

this is my test class:
import org.apache.log4j.Logger;

public class Test
	public static void main(String[] args) throws Exception
		Logger logger = Logger.getRootLogger();
		logger.error("test message");
Comment 1 Curt Arnold 2009-10-07 22:26:31 UTC
Committed fix in rev 823021.

The patch original suggested was sufficient to create the error handler, but not to configure it.  Effective use of error handler required being able to specify the attached logger and appender which the generic reflection code in PropertySetter does not handle.