You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by ck...@apache.org on 2021/09/07 14:33:15 UTC

[logging-log4j2] branch master updated: Pass params to log messages that expect params (#3)

This is an automated email from the ASF dual-hosted git repository.

ckozak pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git


The following commit(s) were added to refs/heads/master by this push:
     new f630e16  Pass params to log messages that expect params (#3)
f630e16 is described below

commit f630e16ecf196b5a7097a224a7997801b819ddd0
Author: Travis Spencer <em...@travisspencer.com>
AuthorDate: Tue Sep 7 15:02:47 2021 +0200

    Pass params to log messages that expect params (#3)
---
 .../main/java/org/apache/logging/log4j/core/script/ScriptManager.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/log4j-core/src/main/java/org/apache/logging/log4j/core/script/ScriptManager.java b/log4j-core/src/main/java/org/apache/logging/log4j/core/script/ScriptManager.java
index f9790af..c9f1830 100644
--- a/log4j-core/src/main/java/org/apache/logging/log4j/core/script/ScriptManager.java
+++ b/log4j-core/src/main/java/org/apache/logging/log4j/core/script/ScriptManager.java
@@ -152,7 +152,7 @@ public class ScriptManager implements FileWatcher, Serializable {
     public void fileModified(final File file) {
         final ScriptRunner runner = scriptRunners.get(file.toString());
         if (runner == null) {
-            logger.info("{} is not a running script");
+            logger.info("{} is not a running script", file.getName());
             return;
         }
         final ScriptEngine engine = runner.getScriptEngine();
@@ -168,7 +168,7 @@ public class ScriptManager implements FileWatcher, Serializable {
     public Object execute(final String name, final Bindings bindings) {
         final ScriptRunner scriptRunner = scriptRunners.get(name);
         if (scriptRunner == null) {
-            logger.warn("No script named {} could be found");
+            logger.warn("No script named {} could be found", name);
             return null;
         }
         return AccessController.doPrivileged((PrivilegedAction<Object>) () -> scriptRunner.execute(bindings));