Java 11 (#1)

* java 11 successful build

* fix CTS cast exception

* add java 11 to build workflow

Author: maximthomas

.github/workflows/build.yml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        java: [ '8']
+        java: [ '8', '11' ]
         os: [ 'ubuntu-latest', 'macos-10.15', 'windows-latest' ]
     steps:
     - uses: actions/checkout@v2
@@ -37,4 +37,4 @@ jobs:
     - name: Build with Maven
       env:
         MAVEN_OPTS: -Dhttps.protocols=TLSv1.2 -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.requestSentEnabled=true -Dmaven.wagon.http.retryHandler.count=10
-      run: mvn --batch-mode --update-snapshots package ${{ steps.maven-profile-flag.outputs.MAVEN_VERIFY_STAGE }} --file pom.xml ${{ steps.maven-profile-flag.outputs.MAVEN_PROFILE_FLAG }}
+      run: mvn --batch-mode --update-snapshots package ${{ steps.maven-profile-flag.outputs.MAVEN_VERIFY_STAGE }} --file pom.xml ${{ steps.maven-profile-flag.outputs.MAVEN_PROFILE_FLAG }}

openam-authentication/openam-auth-oauth2/src/test/java/org/forgerock/openam/authentication/modules/oauth2/ESIATest.java

@@ -12,11 +12,13 @@
 import org.forgerock.openam.authentication.modules.oauth2.service.esia.Signer;
 import org.mockito.Matchers;
 import org.powermock.api.mockito.PowerMockito;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.testng.PowerMockTestCase;
 import org.testng.annotations.Test;
 
 @PrepareForTest(HttpRequestContent.class)
+@PowerMockIgnore("jdk.internal.reflect.*")
 public class ESIATest extends PowerMockTestCase {
 
 	@Test

openam-cassandra/openam-cassandra-embedded/src/test/java/ServerTest.java

@@ -15,7 +15,6 @@
  */
 
 
-
 import org.junit.AfterClass;
 import org.junit.BeforeClass;
 import org.junit.Test;

openam-core/src/main/java/org/forgerock/openam/cts/impl/query/worker/queries/CTSWorkerPastExpiryDateQuery.java

@@ -20,6 +20,7 @@
 
 import javax.inject.Inject;
 import java.util.Calendar;
+import java.util.List;
 
 import org.forgerock.openam.cts.CoreTokenConfig;
 import org.forgerock.openam.sm.datalayer.api.ConnectionFactory;
@@ -28,8 +29,10 @@
 import org.forgerock.openam.sm.datalayer.api.query.QueryBuilder;
 import org.forgerock.openam.sm.datalayer.api.query.QueryFactory;
 import org.forgerock.openam.tokens.CoreTokenField;
+import org.forgerock.opendj.ldap.Filter;
 import org.forgerock.util.Reject;
 import org.forgerock.util.query.QueryFilter;
+import org.forgerock.util.query.QueryFilterVisitor;
 
 /**
  * A query that selects all CTS tokens whose expiry date field is prior to the current timestamp (e.g. who have
@@ -39,7 +42,7 @@
  */
 public class CTSWorkerPastExpiryDateQuery<C> extends CTSWorkerBaseQuery {
 
-    private final QueryFactory<C, CoreTokenField> queryFactory;
+    private final QueryFactory<C, Filter> queryFactory;
     private final int pageSize;
 
     @Inject
@@ -58,8 +61,11 @@ public QueryBuilder getQuery() {
 
         QueryFilter<CoreTokenField> filter = QueryFilter.lessThan(CoreTokenField.EXPIRY_DATE, now);
 
+        QueryFilterVisitor<Filter, Void, CoreTokenField> fc = queryFactory.createFilterConverter();
+
+        Filter accepted = filter.accept(fc, null);
         return queryFactory.createInstance()
-                .withFilter(filter.accept(queryFactory.createFilterConverter(), null))
+                .withFilter(accepted)
                 .pageResultsBy(pageSize)
                 .returnTheseAttributes(CoreTokenField.TOKEN_ID);
     }

openam-core/src/test/java/com/iplanet/dpro/session/service/SessionServerConfigTest.java

@@ -6,6 +6,7 @@
 import com.sun.identity.shared.debug.Debug;
 import org.forgerock.openam.session.SessionServiceURLService;
 import org.powermock.api.mockito.PowerMockito;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.testng.PowerMockTestCase;
 import org.testng.annotations.Test;
@@ -16,6 +17,7 @@
 import static org.mockito.Mockito.mock;
 
 @PrepareForTest({ SystemProperties.class, WebtopNaming.class })
+@PowerMockIgnore("jdk.internal.reflect.*")
 public class SessionServerConfigTest extends PowerMockTestCase {
 
     @Test

openam-core/src/test/java/org/forgerock/openam/cts/impl/LDAPConfigTest.java

@@ -18,6 +18,7 @@
 import com.iplanet.am.util.SystemProperties;
 import org.forgerock.opendj.ldap.DN;
 import org.powermock.api.mockito.PowerMockito;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.testng.PowerMockTestCase;
 import org.testng.annotations.BeforeMethod;
@@ -33,6 +34,7 @@
  * @author [email protected]
  */
 @PrepareForTest(SystemProperties.class)
+@PowerMockIgnore("jdk.internal.reflect.*")
 public class LDAPConfigTest extends PowerMockTestCase {
 
     private LDAPConfig config;

openam-core/src/test/java/org/forgerock/openam/sm/datalayer/impl/ldap/ExternalLdapConfigTest.java

@@ -31,6 +31,7 @@
 import org.forgerock.openam.cts.impl.CTSDataLayerConfiguration;
 import org.forgerock.openam.ldap.LDAPURL;
 import org.powermock.api.mockito.PowerMockito;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.testng.PowerMockTestCase;
 import org.testng.annotations.BeforeMethod;
@@ -41,6 +42,7 @@
 import com.sun.identity.shared.debug.Debug;
 
 @PrepareForTest({ SystemProperties.class, WebtopNaming.class })
+@PowerMockIgnore("jdk.internal.reflect.*")
 public class ExternalLdapConfigTest extends PowerMockTestCase {
 
     private Debug debug;

openam-federation/OpenFM/src/main/java/com/sun/identity/classloader/MaskingClassLoader.java

@@ -30,11 +30,11 @@
 
 import java.util.Collection;
 import java.util.Enumeration;
+import java.util.NoSuchElementException;
 import java.util.Vector;
 import java.io.IOException;
 import java.net.URL;
 import java.net.MalformedURLException;
-import sun.misc.CompoundEnumeration;
 
 /**
  * {@link ClassLoader} that masks a specified set of classes
@@ -252,4 +252,37 @@ public synchronized String toString() {
         return "com.sun.identity.classloader.MaskingClassLoader : Super is : " 
             + super.toString();
     }
+
+    static class CompoundEnumeration<E> implements Enumeration<E> {
+        private final Enumeration<E>[] enums;
+        private int index;
+
+        public CompoundEnumeration(Enumeration<E>[] enums) {
+            this.enums = enums;
+        }
+
+        private boolean next() {
+            while(this.index < this.enums.length) {
+                if (this.enums[this.index] != null && this.enums[this.index].hasMoreElements()) {
+                    return true;
+                }
+
+                ++this.index;
+            }
+
+            return false;
+        }
+
+        public boolean hasMoreElements() {
+            return this.next();
+        }
+
+        public E nextElement() {
+            if (!this.next()) {
+                throw new NoSuchElementException();
+            } else {
+                return this.enums[this.index].nextElement();
+            }
+        }
+    }
 }

openam-federation/openam-federation-library/src/main/java/com/sun/identity/sae/api/SecureAttrs.java

@@ -37,7 +37,6 @@
 import java.io.UnsupportedEncodingException;
 import java.security.MessageDigest;
 import java.security.NoSuchAlgorithmException;
-import sun.misc.CharacterEncoder;
 import com.sun.identity.shared.encode.Base64;
 import com.sun.identity.security.DataEncryptor;
 import java.security.*;

openam-scripting/src/main/java/org/forgerock/openam/scripting/ThreadPoolScriptEvaluator.java

@@ -172,8 +172,8 @@ public void onConfigurationChange(final ScriptEngineConfiguration newConfigurati
                                 newConfiguration.getThreadPoolIdleTimeoutSeconds());
                     }
 
-                    delegateConfigurator.setCorePoolSize(newConfiguration.getThreadPoolCoreSize());
                     delegateConfigurator.setMaximumPoolSize(newConfiguration.getThreadPoolMaxSize());
+                    delegateConfigurator.setCorePoolSize(newConfiguration.getThreadPoolCoreSize());
                     delegateConfigurator.setKeepAliveTime(newConfiguration.getThreadPoolIdleTimeoutSeconds(), TimeUnit.SECONDS);
                 }