You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by rp...@apache.org on 2016/09/13 23:11:03 UTC

logging-log4j2 git commit: LOG4J2-1582 When initializing on platforms where JMX is not available, Interpolator component should not print stack trace for warning messages.

Repository: logging-log4j2
Updated Branches:
  refs/heads/master 789d74ad3 -> 01cc155e7


LOG4J2-1582 When initializing on platforms where JMX is not available, Interpolator component should not print stack trace for warning messages.


Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/01cc155e
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/01cc155e
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/01cc155e

Branch: refs/heads/master
Commit: 01cc155e7577d96f9615e33d895f0953e65c50ac
Parents: 789d74a
Author: rpopma <rp...@apache.org>
Authored: Wed Sep 14 08:11:07 2016 +0900
Committer: rpopma <rp...@apache.org>
Committed: Wed Sep 14 08:11:07 2016 +0900

----------------------------------------------------------------------
 .../org/apache/logging/log4j/core/lookup/Interpolator.java     | 6 +++---
 src/changes/changes.xml                                        | 3 +++
 2 files changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/01cc155e/log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/Interpolator.java
----------------------------------------------------------------------
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 3d4886a..02a0fc6 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
@@ -96,9 +96,9 @@ public class Interpolator extends AbstractConfigurationAwareLookup {
                 Loader.newCheckedInstanceOf("org.apache.logging.log4j.core.lookup.JndiLookup", StrLookup.class));
         } catch (final Throwable e) {
             // java.lang.VerifyError: org/apache/logging/log4j/core/lookup/JndiLookup
-            LOGGER.warn(
-                    "JNDI lookup class is not available because this JRE does not support JNDI. JNDI string lookups will not be available, continuing configuration.",
-                    e);
+            LOGGER.warn( // LOG4J2-1582 don't print the whole stack trace (it is just a warning...)
+                    "JNDI lookup class is not available because this JRE does not support JNDI." +
+                    " JNDI string lookups will not be available, continuing configuration. Ignoring: " + e);
         }
         // JMX input args
         try {

http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/01cc155e/src/changes/changes.xml
----------------------------------------------------------------------
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index 2c4fe2b..7f8bcc3 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -24,6 +24,9 @@
   </properties>
   <body>
     <release version="2.7" date="2016-MM-DD" description="GA Release 2.7">
+      <action issue="LOG4J2-1582" dev="rpopma" type="fix">
+        When initializing on platforms where JMX is not available, Interpolator component no longer prints stack trace for warning messages.
+      </action>
       <action issue="LOG4J2-1313" dev="rpopma" type="fix" due-to="Philipp Knobel, Leon Finker">
         Support Property values to be specified in configuration as a value attribute as well as an element.
       </action>