keycloak-aplcache
Changes
saml/client-adapter/tomcat/tomcat-core/src/main/java/org/keycloak/adapters/saml/AbstractSamlAuthenticatorValve.java 2(+1 -1)
testsuite/jetty/jetty81/pom.xml 8(+4 -4)
testsuite/jetty/pom.xml 21(+21 -0)
testsuite/pom.xml 4(+1 -3)
Details
diff --git a/saml/client-adapter/core/src/main/java/org/keycloak/adapters/saml/SamlAuthenticator.java b/saml/client-adapter/core/src/main/java/org/keycloak/adapters/saml/SamlAuthenticator.java
index f95bb5d..ae7eb4b 100755
--- a/saml/client-adapter/core/src/main/java/org/keycloak/adapters/saml/SamlAuthenticator.java
+++ b/saml/client-adapter/core/src/main/java/org/keycloak/adapters/saml/SamlAuthenticator.java
@@ -270,7 +270,7 @@ public abstract class SamlAuthenticator {
if (attributeValues != null) {
for (Object attrValue : attributeValues) {
String role = getAttributeValue(attrValue);
- log.info("Add role: " + role);
+ log.debugv("Add role: {0}", role);
roles.add(role);
}
}
diff --git a/saml/client-adapter/tomcat/tomcat-core/src/main/java/org/keycloak/adapters/saml/AbstractSamlAuthenticatorValve.java b/saml/client-adapter/tomcat/tomcat-core/src/main/java/org/keycloak/adapters/saml/AbstractSamlAuthenticatorValve.java
index 2032488..af5a8c6 100755
--- a/saml/client-adapter/tomcat/tomcat-core/src/main/java/org/keycloak/adapters/saml/AbstractSamlAuthenticatorValve.java
+++ b/saml/client-adapter/tomcat/tomcat-core/src/main/java/org/keycloak/adapters/saml/AbstractSamlAuthenticatorValve.java
@@ -179,7 +179,7 @@ public abstract class AbstractSamlAuthenticatorValve extends FormAuthenticator i
}
protected boolean authenticateInternal(Request request, HttpServletResponse response, Object loginConfig) throws IOException {
- log.info("authenticateInternal");
+ log.fine("authenticateInternal");
CatalinaHttpFacade facade = new CatalinaHttpFacade(response, request);
SamlDeployment deployment = deploymentContext.resolveDeployment(facade);
if (deployment == null || !deployment.isConfigured()) {
testsuite/jetty/jetty81/pom.xml 8(+4 -4)
diff --git a/testsuite/jetty/jetty81/pom.xml b/testsuite/jetty/jetty81/pom.xml
index 2925c54..d079952 100755
--- a/testsuite/jetty/jetty81/pom.xml
+++ b/testsuite/jetty/jetty81/pom.xml
@@ -27,10 +27,6 @@
<artifactId>keycloak-admin-client</artifactId>
</dependency>
<dependency>
- <groupId>org.keycloak</groupId>
- <artifactId>keycloak-jetty81-saml-adapter</artifactId>
- </dependency>
- <dependency>
<groupId>log4j</groupId>
<artifactId>log4j</artifactId>
</dependency>
@@ -112,6 +108,10 @@
</dependency>
<dependency>
<groupId>org.keycloak</groupId>
+ <artifactId>keycloak-jetty81-saml-adapter</artifactId>
+ </dependency>
+ <dependency>
+ <groupId>org.keycloak</groupId>
<artifactId>keycloak-jetty81-adapter</artifactId>
</dependency>
<dependency>
diff --git a/testsuite/jetty/jetty81/src/test/java/org/keycloak/testsuite/Jetty8Test.java b/testsuite/jetty/jetty81/src/test/java/org/keycloak/testsuite/Jetty8Test.java
index 2453aab..a2e4b42 100755
--- a/testsuite/jetty/jetty81/src/test/java/org/keycloak/testsuite/Jetty8Test.java
+++ b/testsuite/jetty/jetty81/src/test/java/org/keycloak/testsuite/Jetty8Test.java
@@ -87,6 +87,7 @@ public class Jetty8Test {
public static void shutdownJetty() throws Exception {
server.stop();
server.destroy();
+ Thread.sleep(1000);
}
@Rule
diff --git a/testsuite/jetty/jetty81/src/test/java/org/keycloak/testsuite/JettySamlTest.java b/testsuite/jetty/jetty81/src/test/java/org/keycloak/testsuite/JettySamlTest.java
index b64fae5..644435a 100755
--- a/testsuite/jetty/jetty81/src/test/java/org/keycloak/testsuite/JettySamlTest.java
+++ b/testsuite/jetty/jetty81/src/test/java/org/keycloak/testsuite/JettySamlTest.java
@@ -99,6 +99,7 @@ public class JettySamlTest {
public static void shutdownJetty() throws Exception {
server.stop();
server.destroy();
+ Thread.sleep(1000);
}
@Test
diff --git a/testsuite/jetty/jetty91/src/test/java/org/keycloak/testsuite/Jetty9Test.java b/testsuite/jetty/jetty91/src/test/java/org/keycloak/testsuite/Jetty9Test.java
index ba6b2e9..fcf75ca 100755
--- a/testsuite/jetty/jetty91/src/test/java/org/keycloak/testsuite/Jetty9Test.java
+++ b/testsuite/jetty/jetty91/src/test/java/org/keycloak/testsuite/Jetty9Test.java
@@ -87,6 +87,7 @@ public class Jetty9Test {
public static void shutdownJetty() throws Exception {
server.stop();
server.destroy();
+ Thread.sleep(1000);
}
@Rule
diff --git a/testsuite/jetty/jetty91/src/test/java/org/keycloak/testsuite/JettySamlTest.java b/testsuite/jetty/jetty91/src/test/java/org/keycloak/testsuite/JettySamlTest.java
index ac09618..c9323c4 100755
--- a/testsuite/jetty/jetty91/src/test/java/org/keycloak/testsuite/JettySamlTest.java
+++ b/testsuite/jetty/jetty91/src/test/java/org/keycloak/testsuite/JettySamlTest.java
@@ -98,6 +98,7 @@ public class JettySamlTest {
public static void shutdownJetty() throws Exception {
server.stop();
server.destroy();
+ Thread.sleep(1000);
}
@Test
testsuite/jetty/pom.xml 21(+21 -0)
diff --git a/testsuite/jetty/pom.xml b/testsuite/jetty/pom.xml
new file mode 100755
index 0000000..c1f4f4f
--- /dev/null
+++ b/testsuite/jetty/pom.xml
@@ -0,0 +1,21 @@
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+ xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
+ <parent>
+ <artifactId>keycloak-testsuite-pom</artifactId>
+ <groupId>org.keycloak</groupId>
+ <version>1.6.0.Final-SNAPSHOT</version>
+ <relativePath>../pom.xml</relativePath>
+ </parent>
+ <name>Keycloak SAML Jetty Testsuite Integration</name>
+ <description/>
+ <modelVersion>4.0.0</modelVersion>
+
+ <artifactId>keycloak-jetty-testsuite-integration-pom</artifactId>
+ <packaging>pom</packaging>
+
+ <modules>
+ <module>jetty81</module>
+ <module>jetty91</module>
+ <module>jetty92</module>
+ </modules>
+</project>
testsuite/pom.xml 4(+1 -3)
diff --git a/testsuite/pom.xml b/testsuite/pom.xml
index beae79e..5f4274e 100755
--- a/testsuite/pom.xml
+++ b/testsuite/pom.xml
@@ -69,9 +69,7 @@
<module>tomcat6</module>
<module>tomcat7</module>
<module>tomcat8</module>
- <module>jetty/jetty81</module>
- <module>jetty/jetty92</module>
- <module>jetty/jetty91</module>
+ <module>jetty</module>
<module>performance</module>
<!--<module>docker-cluster</module>-->
</modules>