Catch all thrown stuff in AsyncUpdater

To "protect" the fixed rate scheduling from being aborted in ScheduledExecutorService
digipost · Mar 4, 2025 · 5995073 · 5995073
1 parent c62ce4a
commit 5995073
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/main/java/no/digipost/monitoring/async/AsyncUpdater.java b/src/main/java/no/digipost/monitoring/async/AsyncUpdater.java
@@ -18,9 +18,10 @@
 import java.time.Clock;
 import java.time.Duration;
 import java.time.Instant;
-import java.util.logging.Level;
 import java.util.logging.Logger;
 
+import static java.util.logging.Level.WARNING;
+
 class AsyncUpdater implements Runnable {
 
     private static final Logger LOG = Logger.getLogger(AsyncUpdater.class.getName());
@@ -46,8 +47,10 @@ public void run() {
             updateNewValues.run();
             lastUpdate = clock.instant();
             lastUpdateSuccessful = true;
-        } catch (Exception e) {
-            LOG.log(Level.WARNING, "Unexpected exception in updater '" + updaterName + "' while updating metrics.", e);
+        } catch (Throwable e) {
+            LOG.log(WARNING,
+                    "Unexpected exception in updater '" + updaterName + "' while updating metrics: " +
+                    e.getClass().getSimpleName() + " " + e.getMessage(), e);
             lastUpdateSuccessful = false;
         }
     }