Fixed unused field in hiding passwords (strimzi#877)

Signed-off-by: Paolo Patierno <[email protected]>
kornys · Mar 3, 2024 · e1180e4 · e1180e4
1 parent afbf0a3
commit e1180e4
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/src/main/java/io/strimzi/kafka/bridge/config/KafkaConfig.java b/src/main/java/io/strimzi/kafka/bridge/config/KafkaConfig.java
@@ -80,7 +80,7 @@ public static KafkaConfig fromMap(Map<String, Object> map) {
 
  @Override
  public String toString() {
- Map<String, Object> configToString = this.hidingPasswords(this.config);
+ Map<String, Object> configToString = this.hidePasswords();
  return "KafkaConfig(" +
  "config=" + configToString +
  ",consumerConfig=" + this.consumerConfig +
@@ -93,12 +93,10 @@ public String toString() {
  * Hides Kafka related password(s) configuration (i.e. truststore and keystore)
  * by replacing each actual password with [hidden] string
  *
- * @param config configuration where to do the replacing
  * @return updated configuration with hidden password(s)
  */
- private Map<String, Object> hidingPasswords(Map<String, Object> config) {
- Map<String, Object> configToString = new HashMap<>();
- configToString.putAll(this.config);
+ private Map<String, Object> hidePasswords() {
+ Map<String, Object> configToString = new HashMap<>(this.config);
  configToString.entrySet().stream()
  .filter(e -> e.getKey().contains("password"))
  .forEach(e -> e.setValue("[hidden]"));