Add debug messages when getting existing value

Author: KostasTsiounis

closed/src/java.base/share/classes/openj9/internal/security/RestrictedSecurity.java

@@ -1315,11 +1315,17 @@ private void updateProviders(String profileExtensionId, List<String> allInfo) {
         }
 
         private String getExistingValue(String property) {
+            if (debug != null) {
+                debug.println("\tGetting previous value of property: " + property);
+            }
+
             // Look for values from profiles that this one extends.
             String existingValue = profileProperties.get(property);
+            String debugMessage = "\t\tPrevious value from extended profile: ";
 
             // If there is no value, look for non-profile values in java.security file.
             if (existingValue == null) {
+                debugMessage = "\t\tPrevious value from java.security file: ";
                 String propertyKey = null;
                 switch (property) {
                 case "jdkCertpathDisabledAlgorithms":
@@ -1342,6 +1348,11 @@ private String getExistingValue(String property) {
                 }
                 existingValue = securityProps.getProperty(propertyKey);
             }
+
+            if ((debug != null) && existingValue != null) {
+                debug.println(debugMessage + existingValue);
+            }
+
             return existingValue;
         }
 
@@ -1626,7 +1637,8 @@ private boolean setProperty(String property, String propertyKey, List<String> al
 
                         // Take existing value of property into account, if applicable.
                         if (existingValue == null) {
-                            printStackTraceAndExit("Property '" + property + "' does not exist in parent profile. Cannot append.");
+                            printStackTraceAndExit("Property '" + property + "' does not exist in"
+                                    + " parent profile or java.security file. Cannot append.");
                         } else if (existingValue.isBlank()) {
                             newValue = value;
                         } else {
@@ -1640,6 +1652,10 @@ private boolean setProperty(String property, String propertyKey, List<String> al
                         // Remove values from property.
                         value = value.substring(1).trim();
                         if (!value.isBlank()) {
+                            if (existingValue == null) {
+                                printStackTraceAndExit("Property '" + property + "' does not exist in"
+                                    + " parent profile or java.security file. Cannot remove.");
+                            }
                             List<String> existingValues = Stream.of(existingValue.split(","))
                                                                 .map(v -> v.trim())
                                                                 .collect(Collectors.toList());
@@ -1653,7 +1669,8 @@ private boolean setProperty(String property, String propertyKey, List<String> al
                         } else {
                             // Nothing to do. Use existing value of property into account, if available.
                             if (existingValue == null) {
-                                printStackTraceAndExit("Property '" + property + "' does not exist in parent profile. Cannot remove.");
+                                printStackTraceAndExit("Property '" + property + "' does not exist in"
+                                    + " parent profile or java.security file. Cannot remove.");
                             } else if (existingValue.isBlank()) {
                                 newValue = value;
                             } else {