fix: prevent TimerTask errors from cancelling their Timer

Signed-off-by: Fred Bricon <[email protected]>

src/main/java/com/redhat/devtools/intellij/lsp4ij/LanguageServerWrapper.java

@@ -497,7 +497,12 @@ private void startStopTimer() {
         timer.schedule(new TimerTask() {
             @Override
             public void run() {
-                stop();
+                try {
+                    stop();
+                } catch (Throwable t) {
+                    //Need to catch time task exceptions, or it will cancel the timer
+                    LOGGER.error("Failed to stop language server "+LanguageServerWrapper.this.serverDefinition.id, t);
+                }
             }
         }, TimeUnit.SECONDS.toMillis(this.serverDefinition.lastDocumentDisconnectedTimeout));
     }

src/main/java/com/redhat/devtools/intellij/lsp4mp4ij/classpath/ClasspathResourceChangedNotifier.java

@@ -24,6 +24,8 @@
 import com.intellij.openapi.util.Pair;
 import com.intellij.openapi.vfs.VirtualFile;
 import org.jetbrains.annotations.NotNull;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import java.util.HashSet;
 import java.util.Set;
@@ -35,6 +37,8 @@
  */
 public class ClasspathResourceChangedNotifier implements Disposable {
 
+    private static final Logger LOGGER = LoggerFactory.getLogger(ClasspathResourceChangedNotifier.class);
+
     private static final long DEBOUNCE_DELAY = 1000;
 
     private final Project project;
@@ -80,7 +84,12 @@ private void asyncNotifyChanges() {
             debounceTask = new TimerTask() {
                 @Override
                 public void run() {
-                    notifyChanges();
+                    try {
+                        notifyChanges();
+                    } catch (Throwable t) {
+                        //Need to catch time task exceptions, or it will cancel the timer
+                        LOGGER.error("Failed to notify classpath resource change", t);
+                    }
                 }
             };