The `log4j2.messageFactory` property is ignored since 2.23.0

[ppkarwasz]

Description

Due to the changes in version 2.23.0, specifying an alternative default message factory through the log4j2.messageFactory
property does not work any more.

[deepblueli]

hi @ppkarwasz, the default message factory also changed from ReusableMessageFactory to ParameterizedMessageFactory when log4j2.enable.threadlocals is set to true.

Given that log4j2.messageFactory property doesn't work now, is there any other way we can change to use ReusableMessageFactory as the message factory?

[lizongbo]

This new code doesn't make the log4j2.messageFactory configuration work.Please roll back the logic that read the proeprties configuration file earlier.

[lizongbo]

My project with version log4j2 2.23.1 almost caused a serious problem

The following code is the correct implementation for the previous one

    public static final Class<? extends MessageFactory> DEFAULT_MESSAGE_FACTORY_CLASS = createClassForProperty(
            "log4j2.messageFactory", ReusableMessageFactory.class, ParameterizedMessageFactory.class);

[lizongbo]

@vy need open this Issue again.

[lizongbo]

this new code is not support for use log4j-slf4j-impl

log4j-slf4j-impl\src\main\java\org\apache\logging\slf4j\Log4jLogger.java
log4j-slf4j-impl Log4jLogger use src/main/java/org/apache/logging/log4j/simple/SimpleLogger.java
not use src/main/java/org/apache/logging/log4j/core/Logger.java

[ppkarwasz]

@lizongbo,

Did you check the 2.24.0-SNAPSHOT? Volkan's patch is not contained in version 2.23.1.

[lizongbo]

@lizongbo,

Did you check the 2.24.0-SNAPSHOT? Volkan's patch is not contained in version 2.23.1.

Thank fo your reply, When I use 2.24.0-SNAPSHOT from https://repository.apache.org/content/groups/snapshots/
then I ound that wokrs ok,
I'll wait for 2.24.0 to be released as soon as possible.