Clean up the registerd logger handlers when stopping the plugin (#311)

Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
microsoft · Nov 27, 2019 · 9b3e4d1 · 9b3e4d1
1 parent 6208426
commit 9b3e4d1
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/...ugin/src/main/java/com/microsoft/java/debug/plugin/internal/JavaDebuggerServerPlugin.java b/...ugin/src/main/java/com/microsoft/java/debug/plugin/internal/JavaDebuggerServerPlugin.java
@@ -35,6 +35,7 @@ public void start(BundleContext context) throws Exception {
     @Override
     public void stop(BundleContext context) throws Exception {
         logger.info("Stopping " + PLUGIN_ID);
+        LogUtils.cleanupHandlers();
     }
 
 }

diff --git a/...ft.java.debug.plugin/src/main/java/com/microsoft/java/debug/plugin/internal/LogUtils.java b/...ft.java.debug.plugin/src/main/java/com/microsoft/java/debug/plugin/internal/LogUtils.java
@@ -11,6 +11,7 @@
 
 package com.microsoft.java.debug.plugin.internal;
 
+import java.util.logging.Handler;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
@@ -32,6 +33,21 @@ public static void initialize(Level level) {
         logger.setLevel(level);
     }
 
+    /**
+     * Remove the logger handlers registered to the global logger.
+     */
+    public static void cleanupHandlers() {
+        Handler[] handlers = logger.getHandlers();
+        for (Handler handler : handlers) {
+            logger.removeHandler(handler);
+        }
+
+        Handler[] usageHandlers = usageDataLogger.getHandlers();
+        for (Handler handler : usageHandlers) {
+            usageDataLogger.removeHandler(handler);
+        }
+    }
+
     /**
      * Configure log level setting for java debugger.
      *