diff --git a/WebRoot/WEB-INF/web.xml b/WebRoot/WEB-INF/web.xml index ba705d4..7ce0136 100644 --- a/WebRoot/WEB-INF/web.xml +++ b/WebRoot/WEB-INF/web.xml @@ -2,6 +2,11 @@ Enter + + + org.springframework.web.context.ContextLoaderListener diff --git a/build.gradle b/build.gradle index b6a6cf8..25eddf3 100644 --- a/build.gradle +++ b/build.gradle @@ -31,35 +31,40 @@ task wrapper(type: Wrapper) { gretty { servletContainer = 'tomcat8' httpPort = 9080 -// springBoot = true + scanInterval = 0 } repositories { mavenCentral() } +configurations { + all*.exclude module: 'commons-logging' + all*.exclude module: 'log4j' +} + dependencies { - providedCompile 'javax.servlet:servlet-api:2.5' - runtime 'javax.servlet:jstl:1.2' - compile('org.apache.struts:struts2-core:2.3.28.1') { - exclude module:'javassist' - } - compile('org.apache.struts:struts2-spring-plugin:2.3.28.1') { - exclude module:'javassist' - } compile 'org.springframework:spring-context:4.2.6.RELEASE' compile 'org.springframework:spring-web:4.2.6.RELEASE' - compile 'org.springframework:spring-webmvc:4.2.6.RELEASE' compile 'org.springframework:spring-jdbc:4.2.6.RELEASE' compile 'org.springframework:spring-orm:4.2.6.RELEASE' - compile 'org.springframework:spring-tx:4.2.6.RELEASE' compile 'org.springframework:spring-test:4.2.6.RELEASE' compile 'org.hibernate:hibernate-core:4.3.11.Final' + compile('org.apache.struts:struts2-core:2.3.28.1') { + exclude module:'javassist' + } + compile('org.apache.struts:struts2-spring-plugin:2.3.28.1') { + exclude module:'javassist' + } compile 'mysql:mysql-connector-java:5.1.38' - compile 'org.apache.logging.log4j:log4j-core:2.5' + compile 'org.slf4j:slf4j-api:1.7.12' + compile 'ch.qos.logback:logback-core:1.1.3' + compile 'ch.qos.logback:logback-classic:1.1.3' + compile 'org.slf4j:log4j-over-slf4j:1.7.12' + compile 'org.slf4j:jcl-over-slf4j:1.7.12' testCompile 'junit:junit:4.12' } diff --git a/src/main/java/org/fireply/enter/listener/ContextFinalizer.java b/src/main/java/org/fireply/enter/listener/ContextFinalizer.java new file mode 100644 index 0000000..628fd75 --- /dev/null +++ b/src/main/java/org/fireply/enter/listener/ContextFinalizer.java @@ -0,0 +1,183 @@ +package org.fireply.enter.listener; + +import java.lang.reflect.Method; +import java.sql.Driver; +import java.sql.DriverManager; +import java.sql.SQLException; +import java.util.Arrays; +import java.util.Enumeration; +import java.util.List; +import java.util.Set; + +import javax.servlet.ServletContextEvent; +import javax.servlet.ServletContextListener; + +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; + + +/** + * Context finalization to close threads (MySQL memory leak prevention). + * This solution combines the best techniques described in the linked Stack + * Overflow answer. + * @see Tomcat Guice/JDBC Memory Leak + */ +public class ContextFinalizer + implements ServletContextListener { + + private static final Logger LOGGER = + LoggerFactory.getLogger(ContextFinalizer.class); + + /** + * Information for cleaning up a thread. + */ + private class ThreadInfo { + + /** + * Name of the thread's initiating class. + */ + private final String name; + + /** + * Cue identifying the thread. + */ + private final String cue; + + /** + * Name of the method to stop the thread. + */ + private final String stop; + + /** + * Basic constructor. + * @param n Name of the thread's initiating class. + * @param c Cue identifying the thread. + * @param s Name of the method to stop the thread. + */ + ThreadInfo(final String n, final String c, final String s) { + this.name = n; + this.cue = c; + this.stop = s; + } + + /** + * @return the name + */ + public String getName() { + return this.name; + } + + /** + * @return the cue + */ + public String getCue() { + return this.cue; + } + + /** + * @return the stop + */ + public String getStop() { + return this.stop; + } + } + + /** + * List of information on threads required to stop. This list may be + * expanded as necessary. + */ + private List threads = Arrays.asList( + // Special cleanup for MySQL JDBC Connector. + new ThreadInfo( + "com.mysql.jdbc.AbandonedConnectionCleanupThread", //$NON-NLS-1$ + "Abandoned connection cleanup thread", //$NON-NLS-1$ + "shutdown" //$NON-NLS-1$ + ), + new ThreadInfo( + "sun.security.provider.SeedGenerator", //$NON-NLS-1$ + "SeedGenerator Thread", //$NON-NLS-1$ + "no method to stop it" //$NON-NLS-1$ + ) + ); + + @Override + public void contextInitialized(final ServletContextEvent sce) { + LOGGER.debug("After context initialized"); + } + + @Override + public final void contextDestroyed(final ServletContextEvent sce) { + + LOGGER.debug("After context destroyed"); + + // Deregister all drivers. + ClassLoader cl = Thread.currentThread().getContextClassLoader(); + Enumeration drivers = DriverManager.getDrivers(); + while (drivers.hasMoreElements()) { + Driver d = drivers.nextElement(); + try { + if(d.getClass().getClassLoader() == cl) { + DriverManager.deregisterDriver(d); + LOGGER.info("Driver {} deregistered", d); + } + else { + LOGGER.info("Driver {} 没有执行反注册，因为它可能在其他地方使用着", d); + } + } catch (SQLException e) { + LOGGER.warn( + String.format( + "Failed to deregister driver %s", //$NON-NLS-1$ + d + ), + e + ); + } + } + + // Handle remaining threads. + Set threadSet = Thread.getAllStackTraces().keySet(); + Thread[] threadArray = threadSet.toArray(new Thread[threadSet.size()]); + for (Thread t:threadArray) { + for (ThreadInfo i:this.threads) { + if (t.getName().contains(i.getCue())) { + synchronized (t) { + try { + Class cls = Class.forName(i.getName()); + if (cls != null) { + Method mth = cls.getMethod(i.getStop()); + if (mth != null) { + mth.invoke(null); + LOGGER.info( + String.format( + "Connection cleanup thread %s shutdown successfully.", //$NON-NLS-1$ + i.getName() + ) + ); + } + } + } catch (NoSuchMethodException e) { + t.interrupt(); + try { + t.join(); + } catch (InterruptedException ie) { + LOGGER.warn("停止线程 {} 失败", i.getName()); + ie.printStackTrace(); + } + LOGGER.info("停止线程 {} 成功", i.getName()); + } catch (Throwable thr) { + LOGGER.warn( + String.format( + "Failed to shutdown connection cleanup thread %s: ", //$NON-NLS-1$ + i.getName(), + thr.getMessage() + ) + ); + thr.printStackTrace(); + } + } + } + } + } + } + +} diff --git a/src/main/resources/logback.xml b/src/main/resources/logback.xml new file mode 100644 index 0000000..21745dc --- /dev/null +++ b/src/main/resources/logback.xml @@ -0,0 +1,21 @@ + + + + + %d{HH:mm:ss.SSS} [%thread] %-5level %logger{36} - %msg%n + + + + + + + + + + + + + + + +