(maven-build-cache-extension) branch master updated: [MBUILDCACHE-113] java.security.AccessController in java.security has been deprecated and marked for removal (#205)

[ol...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

olamy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-build-cache-extension.git


The following commit(s) were added to refs/heads/master by this push:
     new dfc6cd9  [MBUILDCACHE-113] java.security.AccessController in java.security has been deprecated and marked for removal (#205)
dfc6cd9 is described below

commit dfc6cd93d8040017d3b25a0458eaf005debf9232
Author: Sebastian Tiemann <se...@mailbox.org>
AuthorDate: Fri Apr 25 03:53:26 2025 +0200

    [MBUILDCACHE-113] java.security.AccessController in java.security has been deprecated and marked for removal (#205)
    
    * remove java.security manager related code
    
    - see also JEP 486
    
    * remove java.security manager related code
    
    - see also JEP 486
---
 .../org/apache/maven/buildcache/hash/CloseableBuffer.java     | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/main/java/org/apache/maven/buildcache/hash/CloseableBuffer.java b/src/main/java/org/apache/maven/buildcache/hash/CloseableBuffer.java
index df67df7..7b7c60d 100644
--- a/src/main/java/org/apache/maven/buildcache/hash/CloseableBuffer.java
+++ b/src/main/java/org/apache/maven/buildcache/hash/CloseableBuffer.java
@@ -23,9 +23,7 @@
 import java.nio.ByteBuffer;
 import java.nio.channels.FileChannel;
 import java.nio.channels.FileChannel.MapMode;
-import java.security.PrivilegedAction;
 
-import static java.security.AccessController.doPrivileged;
 import static org.apache.maven.buildcache.hash.ReflectionUtils.getField;
 import static org.apache.maven.buildcache.hash.ReflectionUtils.getMethod;
 
@@ -34,14 +32,16 @@
  */
 public class CloseableBuffer implements AutoCloseable {
 
-    private static final Cleaner CLEANER = doPrivileged((PrivilegedAction<Cleaner>) () -> {
+    private static final Cleaner CLEANER = getJVMDependentCleaner();
+
+    private static Cleaner getJVMDependentCleaner() {
         final String jsv = System.getProperty("java.specification.version", "9");
         if (jsv.startsWith("1.")) {
             return DirectCleaner.isSupported() ? new DirectCleaner() : new NoopCleaner();
         } else {
             return UnsafeCleaner.isSupported() ? new UnsafeCleaner() : new NoopCleaner();
         }
-    });
+    }
 
     public static CloseableBuffer directBuffer(int capacity) {
         return new CloseableBuffer(ByteBuffer.allocateDirect(capacity));
@@ -70,8 +70,7 @@ public ByteBuffer getBuffer() {
 
     @Override
     public void close() {
-        // Java 8: () -> CLEANER.clean(buffer)
-        boolean done = doPrivileged((PrivilegedAction<Boolean>) () -> CLEANER.clean(buffer));
+        boolean done = CLEANER.clean(buffer);
         if (done) {
             buffer = null;
         }