don't guard the parameters here, would disable someone setting them to 0

SeleniumHQ · Mar 9, 2016 · f0c63e0 · f0c63e0
1 parent 7308f86
commit f0c63e0
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/java/server/src/org/openqa/selenium/remote/server/SeleniumServer.java b/java/server/src/org/openqa/selenium/remote/server/SeleniumServer.java
@@ -98,12 +98,8 @@ public void boot() {
     handler.setContextPath("/");
     handler.addServlet(DriverServlet.class, "/wd/hub/*");
 
-    if (browserTimeout > 0) {
-      handler.setInitParameter(DriverServlet.BROWSER_TIMEOUT_PARAMETER, String.valueOf(browserTimeout));
-    }
-    if (sessionTimeout > 0) {
-      handler.setInitParameter(DriverServlet.SESSION_TIMEOUT_PARAMETER, String.valueOf(sessionTimeout));
-    }
+    handler.setInitParameter(DriverServlet.BROWSER_TIMEOUT_PARAMETER, String.valueOf(browserTimeout));
+    handler.setInitParameter(DriverServlet.SESSION_TIMEOUT_PARAMETER, String.valueOf(sessionTimeout));
 
     addRcSupport(handler);