This is an automated email from the ASF dual-hosted git repository. rhauch pushed a commit to branch 1.1 in repository https://gitbox.apache.org/repos/asf/kafka.git
The following commit(s) were added to refs/heads/1.1 by this push: new 83b4876 MINOR: Remove header and key/value converter config value logging (#6660) 83b4876 is described below commit 83b487624b9eb3adcbfe5d065a79342afb28dcaf Author: Chris Egerton <chr...@confluent.io> AuthorDate: Thu May 9 18:35:08 2019 -0700 MINOR: Remove header and key/value converter config value logging (#6660) The debug log lines in the `Plugins` class that log header and key/value converter configurations should be altered as the configurations for these converters may contain secrets that should not be logged in plaintext. Instead, only the keys for these configs are safe to expose. Author: Chris Egerton <ceger...@oberlin.edu> Reviewer: Randall Hauch <rha...@gmail.com> --- .../java/org/apache/kafka/connect/runtime/isolation/Plugins.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/connect/runtime/src/main/java/org/apache/kafka/connect/runtime/isolation/Plugins.java b/connect/runtime/src/main/java/org/apache/kafka/connect/runtime/isolation/Plugins.java index f4cd2ba..017d009 100644 --- a/connect/runtime/src/main/java/org/apache/kafka/connect/runtime/isolation/Plugins.java +++ b/connect/runtime/src/main/java/org/apache/kafka/connect/runtime/isolation/Plugins.java @@ -234,8 +234,8 @@ public class Plugins { // Configure the Converter using only the old configuration mechanism ... String configPrefix = classPropertyName + "."; Map<String, Object> converterConfig = config.originalsWithPrefix(configPrefix); - log.debug("Configuring the {} converter with configuration:{}{}", - isKeyConverter ? "key" : "value", System.lineSeparator(), converterConfig); + log.debug("Configuring the {} converter with configuration keys:{}{}", + isKeyConverter ? "key" : "value", System.lineSeparator(), converterConfig.keySet()); plugin.configure(converterConfig, isKeyConverter); return plugin; } @@ -291,7 +291,7 @@ public class Plugins { String configPrefix = classPropertyName + "."; Map<String, Object> converterConfig = config.originalsWithPrefix(configPrefix); converterConfig.put(ConverterConfig.TYPE_CONFIG, ConverterType.HEADER.getName()); - log.debug("Configuring the header converter with configuration:{}{}", System.lineSeparator(), converterConfig); + log.debug("Configuring the header converter with configuration keys:{}{}", System.lineSeparator(), converterConfig.keySet()); plugin.configure(converterConfig); return plugin; }