Changeset 5840 in josm

Timestamp:

2013-04-11T20:01:30+02:00 (12 years ago)

Author:

Don-vip

Message:

see #8571, #8593 - Hide values of proxy VM arguments for privacy concerns

File:

• trunk/src/org/openstreetmap/josm/actions/ShowStatusReportAction.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/actions/ShowStatusReportAction.java

@@ -9 +9 @@
 import java.awt.event.KeyEvent;
 import java.lang.management.ManagementFactory;
+import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashSet;
 import java.util.List;
+import java.util.ListIterator;
 import java.util.Map;
 import java.util.Map.Entry;
@@ -29 +31 @@
 import org.openstreetmap.josm.tools.Utils;
 
-
 /**
  * @author xeen
@@ -37 +38 @@
  */
 public final class ShowStatusReportAction extends JosmAction {
+    
+    /**
+     * Constructs a new {@code ShowStatusReportAction}
+     */
     public ShowStatusReportAction() {
         super(
@@ -70 +75 @@
         text.append("\n");
         try {
-            List<String> vmArguments = ManagementFactory.getRuntimeMXBean().getInputArguments();
+            // Build a new list of VM parameters to modify it below if needed (default implementation returns an UnmodifiableList instance)
+            List<String> vmArguments = new ArrayList<String>(ManagementFactory.getRuntimeMXBean().getInputArguments());
+            // Hide some parameters for privacy concerns
+            for (ListIterator<String> it = vmArguments.listIterator(); it.hasNext(); ) {
+                String value = it.next();
+                if (value.contains("=") && value.toLowerCase().startsWith("-dproxy")) {
+                    it.set(value.split("=")[0]+"=xxx");
+                }
+            }
             if (!vmArguments.isEmpty()) {
                 text.append("VM arguments: "+ vmArguments.toString().replace("\\\\", "\\"));