From 8a1ea2f69c912e7b8d7bf6966ff85ec4a2ad18df Mon Sep 17 00:00:00 2001 From: Jan Bartel Date: Thu, 16 May 2013 12:18:04 +1000 Subject: [PATCH] 407976 JDBCSessionIdManager potentially leaves server in bad state after startup --- .../server/session/JDBCSessionIdManager.java | 28 ++++++++----------- 1 file changed, 11 insertions(+), 17 deletions(-) diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/session/JDBCSessionIdManager.java b/jetty-server/src/main/java/org/eclipse/jetty/server/session/JDBCSessionIdManager.java index a72c0702b09..1a10ba6852b 100644 --- a/jetty-server/src/main/java/org/eclipse/jetty/server/session/JDBCSessionIdManager.java +++ b/jetty-server/src/main/java/org/eclipse/jetty/server/session/JDBCSessionIdManager.java @@ -516,24 +516,18 @@ public class JDBCSessionIdManager extends AbstractSessionIdManager */ @Override public void doStart() - { - try - { - initializeDatabase(); - prepareTables(); - cleanExpiredSessions(); - super.doStart(); - if (LOG.isDebugEnabled()) - LOG.debug("Scavenging interval = "+getScavengeInterval()+" sec"); - _timer=new Timer("JDBCSessionScavenger", true); - setScavengeInterval(getScavengeInterval()); - } - catch (Exception e) - { - LOG.warn("Problem initialising JettySessionIds table", e); - } + throws Exception + { + initializeDatabase(); + prepareTables(); + cleanExpiredSessions(); + super.doStart(); + if (LOG.isDebugEnabled()) + LOG.debug("Scavenging interval = "+getScavengeInterval()+" sec"); + _timer=new Timer("JDBCSessionScavenger", true); + setScavengeInterval(getScavengeInterval()); } - + /** * Stop the scavenger. */