keycloak-uncached

KEYCLOAK-806 Increase defaults for 'SSO Session Idle Timeout'

10/31/2014 6:20:05 AM

Stian Thorgersen

Commit: 8adad9d

Tree: ef22a6b

Parents: 65362be

Changes

model/api/src/main/java/org/keycloak/models/utils/RepresentationToModel.java 2(+1 -1)

services/src/main/java/org/keycloak/services/managers/ApplianceBootstrap.java 2(+1 -1)

testsuite/integration/src/test/java/org/keycloak/testsuite/oauth/RefreshTokenTest.java 2(+1 -1)

testsuite/integration/src/test/java/org/keycloak/testsuite/saml/SamlBindingTest.java 3(+1 -2)

Details

model/api/src/main/java/org/keycloak/models/utils/RepresentationToModel.java 2(+1 -1)

diff --git a/model/api/src/main/java/org/keycloak/models/utils/RepresentationToModel.java b/model/api/src/main/java/org/keycloak/models/utils/RepresentationToModel.java
index 9610395..7c6e29f 100755
--- a/model/api/src/main/java/org/keycloak/models/utils/RepresentationToModel.java
+++ b/model/api/src/main/java/org/keycloak/models/utils/RepresentationToModel.java
@@ -62,7 +62,7 @@ public class RepresentationToModel {
         else newRealm.setAccessTokenLifespan(300);
 
         if (rep.getSsoSessionIdleTimeout() != null) newRealm.setSsoSessionIdleTimeout(rep.getSsoSessionIdleTimeout());
-        else newRealm.setSsoSessionIdleTimeout(600);
+        else newRealm.setSsoSessionIdleTimeout(1800);
         if (rep.getSsoSessionMaxLifespan() != null) newRealm.setSsoSessionMaxLifespan(rep.getSsoSessionMaxLifespan());
         else newRealm.setSsoSessionMaxLifespan(36000);

services/src/main/java/org/keycloak/services/managers/ApplianceBootstrap.java 2(+1 -1)

diff --git a/services/src/main/java/org/keycloak/services/managers/ApplianceBootstrap.java b/services/src/main/java/org/keycloak/services/managers/ApplianceBootstrap.java
index cf5978f..655b148 100755
--- a/services/src/main/java/org/keycloak/services/managers/ApplianceBootstrap.java
+++ b/services/src/main/java/org/keycloak/services/managers/ApplianceBootstrap.java
@@ -49,7 +49,7 @@ public class ApplianceBootstrap {
         realm.setName(adminRealmName);
         realm.setEnabled(true);
         realm.addRequiredCredential(CredentialRepresentation.PASSWORD);
-        realm.setSsoSessionIdleTimeout(300);
+        realm.setSsoSessionIdleTimeout(1800);
         realm.setAccessTokenLifespan(60);
         realm.setSsoSessionMaxLifespan(36000);
         realm.setAccessCodeLifespan(60);

testsuite/integration/src/test/java/org/keycloak/testsuite/oauth/RefreshTokenTest.java 2(+1 -1)

diff --git a/testsuite/integration/src/test/java/org/keycloak/testsuite/oauth/RefreshTokenTest.java b/testsuite/integration/src/test/java/org/keycloak/testsuite/oauth/RefreshTokenTest.java
index fcd51b3..ca4b7ee 100755
--- a/testsuite/integration/src/test/java/org/keycloak/testsuite/oauth/RefreshTokenTest.java
+++ b/testsuite/integration/src/test/java/org/keycloak/testsuite/oauth/RefreshTokenTest.java
@@ -139,7 +139,7 @@ public class RefreshTokenTest {
 
         Assert.assertThat(token.getExpiration() - Time.currentTime(), allOf(greaterThanOrEqualTo(250), lessThanOrEqualTo(300)));
         int actual = refreshToken.getExpiration() - Time.currentTime();
-        Assert.assertThat(actual, allOf(greaterThanOrEqualTo(559), lessThanOrEqualTo(600)));
+        Assert.assertThat(actual, allOf(greaterThanOrEqualTo(1799), lessThanOrEqualTo(1800)));
 
         Assert.assertEquals(sessionId, refreshToken.getSessionState());

testsuite/integration/src/test/java/org/keycloak/testsuite/saml/SamlBindingTest.java 3(+1 -2)

diff --git a/testsuite/integration/src/test/java/org/keycloak/testsuite/saml/SamlBindingTest.java b/testsuite/integration/src/test/java/org/keycloak/testsuite/saml/SamlBindingTest.java
index 302fc91..6c104fb 100755
--- a/testsuite/integration/src/test/java/org/keycloak/testsuite/saml/SamlBindingTest.java
+++ b/testsuite/integration/src/test/java/org/keycloak/testsuite/saml/SamlBindingTest.java
@@ -71,8 +71,7 @@ public class SamlBindingTest {
     @WebResource
     protected LoginPage loginPage;
 
-    @Test
-    @Ignore
+    //@Test
     public void runit() throws Exception {
         Thread.sleep(10000000);
     }