From: Pawel <36118444+s-pw@users.noreply.github.com>
Date: Wed, 16 Feb 2022 06:30:39 +0000 (-0800)
Subject: [bluetooth.airthings] Catch all exceptions in a scheduled task to prevent future... 
X-Git-Url: https://git.basschouten.com/?a=commitdiff_plain;h=85866c63c0705c53113d1663110d3e2faafb171e;p=openhab-addons.git

[bluetooth.airthings] Catch all exceptions in a scheduled task to prevent future executions from being suppressed (#12261)

* Fixes #9862

Catch all exceptions in scheduled task to prevent future executions from being suppressed.

Signed-off-by: Pawel Switalski <switalski.pw@gmail.com>
Signed-off-by: Pawel <s-pw@example.com>
---

diff --git a/bundles/org.openhab.binding.bluetooth.airthings/src/main/java/org/openhab/binding/bluetooth/airthings/internal/AbstractAirthingsHandler.java b/bundles/org.openhab.binding.bluetooth.airthings/src/main/java/org/openhab/binding/bluetooth/airthings/internal/AbstractAirthingsHandler.java
index 4d3d89c3de..464e99fcb4 100644
--- a/bundles/org.openhab.binding.bluetooth.airthings/src/main/java/org/openhab/binding/bluetooth/airthings/internal/AbstractAirthingsHandler.java
+++ b/bundles/org.openhab.binding.bluetooth.airthings/src/main/java/org/openhab/binding/bluetooth/airthings/internal/AbstractAirthingsHandler.java
@@ -106,8 +106,12 @@ abstract public class AbstractAirthingsHandler extends BeaconBluetoothHandler {
     }
 
     private void executePeridioc() {
-        sinceLastReadSec.addAndGet(CHECK_PERIOD_SEC);
-        execute();
+        try {
+            sinceLastReadSec.addAndGet(CHECK_PERIOD_SEC);
+            execute();
+        } catch (Exception e) { // catch all to avoid scheduleWithFixedDelay being suppressed
+            logger.warn("Failed to read Airthings device", e);
+        }
     }
 
     private synchronized void execute() {