From 6105bc99e2af49e7193ce10a41096b57ab1d8c94 Mon Sep 17 00:00:00 2001 From: Kevin Doran Date: Mon, 8 Jan 2018 22:28:08 -0500 Subject: [PATCH] NIFI-4708 Fixes encrypt-config log4j configuration --- .../nifi/toolkit/encryptconfig/EncryptConfigLogger.groovy | 5 ++++- .../src/test/resources/log4j.properties | 1 + 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/nifi-toolkit/nifi-toolkit-encrypt-config/src/main/groovy/org/apache/nifi/toolkit/encryptconfig/EncryptConfigLogger.groovy b/nifi-toolkit/nifi-toolkit-encrypt-config/src/main/groovy/org/apache/nifi/toolkit/encryptconfig/EncryptConfigLogger.groovy index 07c95777a91d..98a09448a8f4 100644 --- a/nifi-toolkit/nifi-toolkit-encrypt-config/src/main/groovy/org/apache/nifi/toolkit/encryptconfig/EncryptConfigLogger.groovy +++ b/nifi-toolkit/nifi-toolkit-encrypt-config/src/main/groovy/org/apache/nifi/toolkit/encryptconfig/EncryptConfigLogger.groovy @@ -37,7 +37,7 @@ class EncryptConfigLogger { static configureLogger(boolean verboseEnabled) { Properties log4jProps = null - URL log4jPropsPath = this.getClass().getResource("log4j.properties") + URL log4jPropsPath = EncryptConfigLogger.class.getResource("/log4j.properties") if (log4jPropsPath) { try { log4jPropsPath.withReader { reader -> @@ -53,6 +53,9 @@ class EncryptConfigLogger { log4jProps = defaultProperties() } + // For encrypt-config, log output should go to System.err as System.out is used for tool output in decrypt mode + log4jProps.put("log4j.appender.console.Target", "System.err") + if (verboseEnabled) { // Override the log level for this package. For this to work as intended, this class must belong // to the same package (or a parent package) of all the encrypt-config classes diff --git a/nifi-toolkit/nifi-toolkit-encrypt-config/src/test/resources/log4j.properties b/nifi-toolkit/nifi-toolkit-encrypt-config/src/test/resources/log4j.properties index aa42a320895d..d68cca08eb09 100644 --- a/nifi-toolkit/nifi-toolkit-encrypt-config/src/test/resources/log4j.properties +++ b/nifi-toolkit/nifi-toolkit-encrypt-config/src/test/resources/log4j.properties @@ -18,6 +18,7 @@ log4j.rootLogger=DEBUG,console,test log4j.appender.console=org.apache.log4j.ConsoleAppender +log4j.appender.console.Target=System.err log4j.appender.console.layout=org.apache.log4j.PatternLayout log4j.appender.console.layout.ConversionPattern=%d{yy/MM/dd HH:mm:ss} %p %c{2}: %m%n