(knox) branch master updated: KNOX-3221: Remove JDK9+ checks from BCInterceptingSocket (#1115)

[hanicz]

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

hanicz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/knox.git


The following commit(s) were added to refs/heads/master by this push:
     new 2752fdd8c KNOX-3221: Remove JDK9+ checks from BCInterceptingSocket 
(#1115)
2752fdd8c is described below

commit 2752fdd8c09ac78d06df3219e956578ab7c52bb1
Author: hanicz <han...@users.noreply.github.com>
AuthorDate: Thu Nov 27 11:00:34 2025 +0100

    KNOX-3221: Remove JDK9+ checks from BCInterceptingSocket (#1115)
---
 .../knox/gateway/fips/BCInterceptingSocket.java    | 31 +++++-----------------
 1 file changed, 6 insertions(+), 25 deletions(-)

diff --git 
a/gateway-spi/src/main/java/org/apache/knox/gateway/fips/BCInterceptingSocket.java
 
b/gateway-spi/src/main/java/org/apache/knox/gateway/fips/BCInterceptingSocket.java
index 636607565..4284beae9 100644
--- 
a/gateway-spi/src/main/java/org/apache/knox/gateway/fips/BCInterceptingSocket.java
+++ 
b/gateway-spi/src/main/java/org/apache/knox/gateway/fips/BCInterceptingSocket.java
@@ -20,7 +20,6 @@ package org.apache.knox.gateway.fips;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
-import java.lang.reflect.InvocationTargetException;
 import java.net.InetAddress;
 import java.net.Socket;
 import java.net.SocketAddress;
@@ -239,37 +238,19 @@ public class BCInterceptingSocket extends Socket {
         delegate.setPerformancePreferences(connectionTime, latency, bandwidth);
     }
 
-    /**
-     * This method is only available in JDK9+ therefor reflection is used to 
call it.
-     */
-    @SuppressWarnings({"PMD.MissingOverride", "unchecked"})
+    @Override
     public Set<SocketOption<?>> supportedOptions() {
-        return invokeDelegateMethod("supportedOptions", new Class<?>[]{});
+        return delegate.supportedOptions();
     }
 
-    /**
-     * This method is only available in JDK9+ therefor reflection is used to 
call it.
-     */
-    @SuppressWarnings({"PMD.MissingOverride", "unchecked"})
+    @Override
     public <T> T getOption(SocketOption<T> name) throws IOException {
-        return invokeDelegateMethod("getOption", new 
Class<?>[]{SocketOption.class}, name);
+        return delegate.getOption(name);
     }
 
 
-    /**
-     * This method is only available in JDK9+ therefor reflection is used to 
call it.
-     */
-    @SuppressWarnings("PMD.MissingOverride")
+    @Override
     public <T> Socket setOption(SocketOption<T> name, T value) throws 
IOException {
-        return invokeDelegateMethod("setOption", new 
Class<?>[]{SocketOption.class, Object.class}, name, value);
-    }
-
-    @SuppressWarnings("unchecked")
-    private <T> T invokeDelegateMethod(String methodName, Class<?>[] 
parameterTypes, Object... args) {
-        try {
-            return (T) delegate.getClass().getMethod(methodName, 
parameterTypes).invoke(delegate, args);
-        } catch (NoSuchMethodException | IllegalAccessException | 
InvocationTargetException e) {
-            throw new UnsupportedOperationException("Socket option not 
supported", e);
-        }
+        return delegate.setOption(name, value);
     }
 }