Never mind, that hard reference is from 2 days ago... BUT... if someone
decides to apply this command to a current version, not so good.

Gary

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

> 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