Illegal reflective access by com.sun.mail.util.SocketFetcher eclipse-…

…ee4j#124 Co-authored-by: jmehrens <[email protected]> Co-authored-by: icu5545 <[email protected]> Signed-off-by: jmehrens <[email protected]>
jmehrens · Feb 8, 2024 · 3c40a82 · 3c40a82
1 parent 04c88b6
commit 3c40a82
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/core/src/main/java/org/eclipse/angus/mail/util/SocketFetcher.java b/core/src/main/java/org/eclipse/angus/mail/util/SocketFetcher.java
@@ -1102,8 +1102,9 @@ public boolean verify(String server, SSLSession ssls) {
      * or --add-opens 'java.base/sun.security.util=jakarta.mail' depending on
      * how this class has been packaged.
      * It is preferred to set mail.<protocol>.ssl.endpointidentitycheck property
-     * to 'LDAPS' instead of using this validator.  This adapter will be removed
-     * in a future release of Angus Mail.
+     * to 'LDAPS' instead of using this verifier.  This adapter will be removed
+     * in a future release of Angus Mail when there is no reason to keep this
+     * for compatiblity sake.
      *
      * See: JDK-8062515 - Migrate use of sun.security.** to supported API
      */

diff --git a/providers/angus-mail/src/test/java/org/eclipse/angus/mail/util/SocketFetcherTest.java b/providers/angus-mail/src/test/java/org/eclipse/angus/mail/util/SocketFetcherTest.java
@@ -349,7 +349,7 @@ public static class TestHostnameVerifier implements HostnameVerifier {
          * the unit test may synchronize on the TestHostnameVerifier class and call the
          * static "reset" method prior to de-synchronizing.
          */
-        public static final AtomicInteger defaultConstructorCount = new AtomicInteger();
+        private static final AtomicInteger defaultConstructorCount = new AtomicInteger();
         private boolean acceptConnections = true;
         private boolean verified = false;