Wouldn't this be better:

diff --git
a/log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/Interpolator.java
b/log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/Interpolator.java
index 75c0a45..9c491ac 100644
---
a/log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/Interpolator.java
+++
b/log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/Interpolator.java
@@ -78,7 +78,7 @@
         for (final Map.Entry<String, PluginType<?>> entry :
plugins.entrySet()) {
             try {
                 final Class<? extends StrLookup> clazz =
entry.getValue().getPluginClass().asSubclass(StrLookup.class);
-                if (!clazz.getName().equals(JndiLookup.class.getName()) ||
JndiManager.isJndiEnabled()) {
+                if
(!clazz.getName().equals("org.apache.logging.log4j.core.lookup.JndiLookup")
|| JndiManager.isJndiEnabled()) {
                     strLookupMap.put(entry.getKey().toLowerCase(),
ReflectionUtil.instantiate(clazz));
                 }
             } catch (final Throwable t) {

?

On Mon, Dec 13, 2021 at 2:48 PM Gary Gregory <[email protected]> wrote:

> zip -q -d log4j-core-*.jar
> org/apache/logging/log4j/core/lookup/JndiLookup.class
>
> This can't be right, can it?
>
> We have a hard reference to that class
> in org.apache.logging.log4j.core.lookup.Interpolator
>
> Should we really recommend this?
>
> Gary
>

Reply via email to