Changeset 19264 in josm for trunk

Timestamp:

2024-12-02T15:29:07+01:00 (26 hours ago)

Author:

taylor.smock

Message:

Fix #24030: JEP 486 (in Java 24) always throws an UnsupportedOperationException when attempting to set security policy

This merely fixes startup issues. There are a few other locations which may
cause issues, but did not during relatively minimal testing.

File:

• trunk/src/org/openstreetmap/josm/gui/MainApplication.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/MainApplication.java

@@ -787 +787 @@
         I18n.set(language.orElse(null));
 
-        try {
-            Policy.setPolicy(new Policy() {
-                // Permissions for plug-ins loaded when josm is started via webstart
-                private final PermissionCollection pc;
-
-                {
-                    pc = new Permissions();
-                    pc.add(new AllPermission());
-                }
-
-                @Override
-                public PermissionCollection getPermissions(CodeSource codesource) {
-                    return pc;
-                }
-            });
-        } catch (SecurityException e) {
-            Logging.log(Logging.LEVEL_ERROR, "Unable to set permissions", e);
+        // JEP 486 (Java 24) now causes exceptions to be thrown. The security manager is terminally deprecated.
+        if (Utils.getJavaVersion() < 24) {
+            try {
+                Policy.setPolicy(new Policy() {
+                    // Permissions for plug-ins loaded when josm is started via webstart
+                    private final PermissionCollection pc;
+
+                    {
+                        pc = new Permissions();
+                        pc.add(new AllPermission());
+                    }
+
+                    @Override
+                    public PermissionCollection getPermissions(CodeSource codesource) {
+                        return pc;
+                    }
+                });
+            } catch (SecurityException e) {
+                Logging.log(Logging.LEVEL_ERROR, "Unable to set permissions", e);
+            }
         }