Return the default enum if a SecurityException is caught in

getEnumSystemProperty()

Author: garydgregory

src/changes/changes.xml

@@ -98,6 +98,7 @@ The <action> type attribute can be add,update,fix,remove.
     <action                   type="fix" dev="ggregory" due-to="Gary Gregory">Fix edge-case NullPointerException in org.apache.commons.lang3.SystemUtils.IS_OS_ANDROID.</action>
     <action                   type="fix" dev="ggregory" due-to="Gary Gregory">Fix edge-case NullPointerException in org.apache.commons.lang3.SystemUtils.isJavaVersionAtLeast(JavaVersion).</action>
     <action                   type="fix" dev="ggregory" due-to="Gary Gregory">Fix edge-case NullPointerException in org.apache.commons.lang3.SystemUtils.isJavaVersionAtMost(JavaVersion).</action>
+    <action                   type="fix" dev="ggregory" due-to="Gary Gregory">Return the default enum if a SecurityException is caught in getEnumSystemProperty().</action>
     <!-- ADD -->
     <action                   type="add" dev="ggregory" due-to="Gary Gregory">Add Strings and refactor StringUtils.</action>
     <action issue="LANG-1747" type="add" dev="ggregory" due-to="Oliver B. Fischer, Gary Gregory">Add StopWatch.run([Failable]Runnable) and get([Failable]Supplier).</action>

src/main/java/org/apache/commons/lang3/EnumUtils.java

@@ -309,18 +309,21 @@ public static <E extends Enum<E>, K> Map<K, E> getEnumMap(final Class<E> enumCla
      * <p>
      * This method differs from {@link Enum#valueOf} in that it does not throw an exception for an invalid enum name.
      * </p>
+     * <p>
+     * If a {@link SecurityException} is caught, the return value is {@code null}.
+     * </p>
      *
-     * @param <E> the type of the enumeration
-     * @param enumClass the class of the enum to query, not null
-     * @param propName the system property key for the enum name, null returns default enum
-     * @param defaultEnum the default enum
-     * @return the enum, default enum if not found
+     * @param <E>         the type of the enumeration.
+     * @param enumClass   the class of the enum to query, not null.
+     * @param propName    the system property key for the enum name, null returns default enum.
+     * @param defaultEnum the default enum.
+     * @return the enum, default enum if not found.
      * @since 3.13.0
      */
     public static <E extends Enum<E>> E getEnumSystemProperty(final Class<E> enumClass, final String propName,
         final E defaultEnum) {
         return enumClass == null || propName == null ? defaultEnum
-            : getEnum(enumClass, System.getProperty(propName), defaultEnum);
+            : getEnum(enumClass, SystemProperties.getProperty(propName), defaultEnum);
     }
 
     /**