diff --git a/source/java/org/alfresco/filesys/AbstractServerConfigurationBean.java b/source/java/org/alfresco/filesys/AbstractServerConfigurationBean.java index 39871651a4..3dd26fbc24 100644 --- a/source/java/org/alfresco/filesys/AbstractServerConfigurationBean.java +++ b/source/java/org/alfresco/filesys/AbstractServerConfigurationBean.java @@ -130,7 +130,7 @@ public abstract class AbstractServerConfigurationBean extends ServerConfiguratio // Maximum session timeout - protected static final int MaxSessionTimeout = 60 * 60; // 1 hour + public static final int MaxSessionTimeout = 60 * 60; // 1 hour // Disk interface to use for shared filesystems diff --git a/source/java/org/alfresco/filesys/config/CIFSConfigBean.java b/source/java/org/alfresco/filesys/config/CIFSConfigBean.java index 9424a620be..5a4d85a2a0 100644 --- a/source/java/org/alfresco/filesys/config/CIFSConfigBean.java +++ b/source/java/org/alfresco/filesys/config/CIFSConfigBean.java @@ -18,9 +18,12 @@ */ package org.alfresco.filesys.config; +import org.alfresco.error.AlfrescoRuntimeException; import org.alfresco.jlan.server.auth.ICifsAuthenticator; import org.alfresco.jlan.smb.server.VirtualCircuitList; +import static org.alfresco.filesys.AbstractServerConfigurationBean.MaxSessionTimeout; + // TODO: Auto-generated Javadoc /** * The Class CIFSConfigBean. @@ -520,9 +523,22 @@ public class CIFSConfigBean */ public void setSessionTimeout(Integer sessionTimeout) { + validateSessionTimeout(sessionTimeout); this.sessionTimeout = sessionTimeout; } + /** + * Validates the session timeout. + * + * @param sessionTimeout + * the session timeout to validate + */ + public void validateSessionTimeout(Integer sessionTimeout) + { + if (sessionTimeout < 0 || sessionTimeout > MaxSessionTimeout) + throw new AlfrescoRuntimeException("Session timeout out of range (0 - " + MaxSessionTimeout + ")"); + } + /** * Set the maximum virtual circuits per session * diff --git a/source/java/org/alfresco/filesys/config/ServerConfigurationBean.java b/source/java/org/alfresco/filesys/config/ServerConfigurationBean.java index 019a311974..934920e8b5 100644 --- a/source/java/org/alfresco/filesys/config/ServerConfigurationBean.java +++ b/source/java/org/alfresco/filesys/config/ServerConfigurationBean.java @@ -1123,8 +1123,7 @@ public class ServerConfigurationBean extends AbstractServerConfigurationBean imp // Validate the session timeout value - if (tmo < 0 || tmo > MaxSessionTimeout) - throw new AlfrescoRuntimeException("Session timeout out of range (0 - " + MaxSessionTimeout + ")"); + cifsConfigBean.validateSessionTimeout(tmo); // Convert the session timeout to milliseconds