Update password logging replacement in ProcessRunner

YLChen-007 · web-flow · commit 165f7b49ed53 · 2025-11-11T10:29:34.000+08:00
diff --git a/utils/src/main/java/org/apache/cloudstack/utils/process/ProcessRunner.java b/utils/src/main/java/org/apache/cloudstack/utils/process/ProcessRunner.java
@@ -67,7 +67,7 @@ String removeCommandSensitiveInfoForLogging(String command) {
     public ProcessRunner(ExecutorService executor) {
         this.executor = executor;
         commandLogReplacements.add(new Ternary<>("ipmitool", "-P\\s+\\S+", "-P *****"));
-        commandLogReplacements.add(new Ternary<>("ipmitool", "(?i)(password)(\\s+)\\S+(\\s+)\\S+", "$1****$2****"));
+        commandLogReplacements.add(new Ternary<>("ipmitool", "(?i)(password)(\\s+)\\S+(\\s+)\\S+", "$1 ****$2****"));
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ String removeCommandSensitiveInfoForLogging(String command) {`
`67`	`67`	`public ProcessRunner(ExecutorService executor) {`
`68`	`68`	`this.executor = executor;`
`69`	`69`	`commandLogReplacements.add(new Ternary<>("ipmitool", "-P\\s+\\S+", "-P *****"));`
`70`		`- commandLogReplacements.add(new Ternary<>("ipmitool", "(?i)(password)(\\s+)\\S+(\\s+)\\S+", "$1**$2**"));`
	`70`	`+ commandLogReplacements.add(new Ternary<>("ipmitool", "(?i)(password)(\\s+)\\S+(\\s+)\\S+", "$1 **$2**"));`
`71`	`71`	`}`
`72`	`72`
`73`	`73`	`/**`