Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2801#discussion_r209438801 --- Diff: storm-client/src/jvm/org/apache/storm/utils/ConfigUtils.java --- @@ -52,6 +79,16 @@ public static ConfigUtils setInstance(ConfigUtils u) { return oldInstance; } + public static Map<String, Object> maskPasswords(final Map<String, Object> conf) { + Maps.EntryTransformer<String, Object, Object> maskPasswords = --- End diff -- @arunmahadevan I also think this approach can replace `Utils.radactValue` via adding `@password` to STORM_ZOOKEEPER_TOPOLOGY_AUTH_PAYLOAD. What do you think? Looks like we don't need to have both approach.
---