[bindings a-c] Fix exception handling (Jetty HTTP client) (#10467)

Signed-off-by: Laurent Garnier <lg.hc@free.fr>

bundles/org.openhab.binding.autelis/src/main/java/org/openhab/binding/autelis/internal/handler/AutelisHandler.java

@@ -18,8 +18,10 @@ import java.util.Base64;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.Map;
+import java.util.concurrent.ExecutionException;
 import java.util.concurrent.ScheduledFuture;
 import java.util.concurrent.TimeUnit;
+import java.util.concurrent.TimeoutException;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
@@ -190,88 +192,94 @@ public class AutelisHandler extends BaseThingHandler {
 
     @Override
     public void handleCommand(ChannelUID channelUID, Command command) {
-        logger.debug("handleCommand channel: {} command: {}", channelUID.getId(), command);
+        try {
-        if (AutelisBindingConstants.CMD_LIGHTS.equals(channelUID.getId())) {
+            logger.debug("handleCommand channel: {} command: {}", channelUID.getId(), command);
-            /*
+            if (AutelisBindingConstants.CMD_LIGHTS.equals(channelUID.getId())) {
-             * lighting command possible values, but we will let anything
+                /*
-             * through. alloff, allon, csync, cset, cswim, party, romance,
+                 * lighting command possible values, but we will let anything
-             * caribbean, american, sunset, royalty, blue, green, red, white,
+                 * through. alloff, allon, csync, cset, cswim, party, romance,
-             * magenta, hold, recall
+                 * caribbean, american, sunset, royalty, blue, green, red, white,
-             */
+                 * magenta, hold, recall
-            getUrl(baseURL + "/lights.cgi?val=" + command.toString(), TIMEOUT_SECONDS);
+                 */
-        } else if (AutelisBindingConstants.CMD_REBOOT.equals(channelUID.getId()) && command == OnOffType.ON) {
+                getUrl(baseURL + "/lights.cgi?val=" + command.toString(), TIMEOUT_SECONDS);
-            getUrl(baseURL + "/userreboot.cgi?do=true" + command.toString(), TIMEOUT_SECONDS);
+            } else if (AutelisBindingConstants.CMD_REBOOT.equals(channelUID.getId()) && command == OnOffType.ON) {
-            updateState(channelUID, OnOffType.OFF);
+                getUrl(baseURL + "/userreboot.cgi?do=true" + command.toString(), TIMEOUT_SECONDS);
-        } else {
+                updateState(channelUID, OnOffType.OFF);
-            String[] args = channelUID.getId().split("-");
-            if (args.length < 2) {
-                logger.warn("Unown channel {} for command {}", channelUID, command);
-                return;
-            }
-            String type = args[0];
-            String name = args[1];
-
-            if (AutelisBindingConstants.CMD_EQUIPMENT.equals(type)) {
-                String cmd = "value";
-                int value;
-                if (command == OnOffType.OFF) {
-                    value = 0;
-                } else if (command == OnOffType.ON) {
-                    value = 1;
-                } else if (command instanceof DecimalType) {
-                    value = ((DecimalType) command).intValue();
-                    if (!isJandy() && value >= 3) {
-                        // this is a autelis dim type. not sure what 2 does
-                        cmd = "dim";
-                    }
-                } else {
-                    logger.error("command type {} is not supported", command);
-                    return;
-                }
-                String response = getUrl(baseURL + "/set.cgi?name=" + name + "&" + cmd + "=" + value, TIMEOUT_SECONDS);
-                logger.debug("equipment set {} {} {} : result {}", name, cmd, value, response);
-            } else if (AutelisBindingConstants.CMD_TEMP.equals(type)) {
-                String value;
-                if (command == IncreaseDecreaseType.INCREASE) {
-                    value = "up";
-                } else if (command == IncreaseDecreaseType.DECREASE) {
-                    value = "down";
-                } else if (command == OnOffType.OFF) {
-                    value = "0";
-                } else if (command == OnOffType.ON) {
-                    value = "1";
-                } else {
-                    value = command.toString();
-                }
-
-                String cmd;
-                // name ending in sp are setpoints, ht are heater?
-                if (name.endsWith("sp")) {
-                    cmd = "temp";
-                } else if (name.endsWith("ht")) {
-                    cmd = "hval";
-                } else {
-                    logger.error("Unknown temp type {}", name);
-                    return;
-                }
-                String response = getUrl(baseURL + "/set.cgi?wait=1&name=" + name + "&" + cmd + "=" + value,
-                        TIMEOUT_SECONDS);
-                logger.debug("temp set name:{} cmd:{} value:{} : result {}", name, cmd, value, response);
-            } else if (AutelisBindingConstants.CMD_CHEM.equals(type)) {
-                String response = getUrl(baseURL + "/set.cgi?name=" + name + "&chem=" + command.toString(),
-                        TIMEOUT_SECONDS);
-                logger.debug("chlrp {} {}: result {}", name, command, response);
-            } else if (AutelisBindingConstants.CMD_PUMPS.equals(type)) {
-                String response = getUrl(baseURL + "/set.cgi?name=" + name + "&speed=" + command.toString(),
-                        TIMEOUT_SECONDS);
-                logger.debug("pumps {} {}: result {}", name, command, response);
             } else {
-                logger.error("Unsupported type {}", type);
+                String[] args = channelUID.getId().split("-");
+                if (args.length < 2) {
+                    logger.warn("Unown channel {} for command {}", channelUID, command);
+                    return;
+                }
+                String type = args[0];
+                String name = args[1];
+
+                if (AutelisBindingConstants.CMD_EQUIPMENT.equals(type)) {
+                    String cmd = "value";
+                    int value;
+                    if (command == OnOffType.OFF) {
+                        value = 0;
+                    } else if (command == OnOffType.ON) {
+                        value = 1;
+                    } else if (command instanceof DecimalType) {
+                        value = ((DecimalType) command).intValue();
+                        if (!isJandy() && value >= 3) {
+                            // this is a autelis dim type. not sure what 2 does
+                            cmd = "dim";
+                        }
+                    } else {
+                        logger.error("command type {} is not supported", command);
+                        return;
+                    }
+                    String response = getUrl(baseURL + "/set.cgi?name=" + name + "&" + cmd + "=" + value,
+                            TIMEOUT_SECONDS);
+                    logger.debug("equipment set {} {} {} : result {}", name, cmd, value, response);
+                } else if (AutelisBindingConstants.CMD_TEMP.equals(type)) {
+                    String value;
+                    if (command == IncreaseDecreaseType.INCREASE) {
+                        value = "up";
+                    } else if (command == IncreaseDecreaseType.DECREASE) {
+                        value = "down";
+                    } else if (command == OnOffType.OFF) {
+                        value = "0";
+                    } else if (command == OnOffType.ON) {
+                        value = "1";
+                    } else {
+                        value = command.toString();
+                    }
+
+                    String cmd;
+                    // name ending in sp are setpoints, ht are heater?
+                    if (name.endsWith("sp")) {
+                        cmd = "temp";
+                    } else if (name.endsWith("ht")) {
+                        cmd = "hval";
+                    } else {
+                        logger.error("Unknown temp type {}", name);
+                        return;
+                    }
+                    String response = getUrl(baseURL + "/set.cgi?wait=1&name=" + name + "&" + cmd + "=" + value,
+                            TIMEOUT_SECONDS);
+                    logger.debug("temp set name:{} cmd:{} value:{} : result {}", name, cmd, value, response);
+                } else if (AutelisBindingConstants.CMD_CHEM.equals(type)) {
+                    String response = getUrl(baseURL + "/set.cgi?name=" + name + "&chem=" + command.toString(),
+                            TIMEOUT_SECONDS);
+                    logger.debug("chlrp {} {}: result {}", name, command, response);
+                } else if (AutelisBindingConstants.CMD_PUMPS.equals(type)) {
+                    String response = getUrl(baseURL + "/set.cgi?name=" + name + "&speed=" + command.toString(),
+                            TIMEOUT_SECONDS);
+                    logger.debug("pumps {} {}: result {}", name, command, response);
+                } else {
+                    logger.error("Unsupported type {}", type);
+                }
             }
+            clearState(true);
+            // reset the schedule for our next poll which at that time will reflect if our command was successful or
+            // not.
+            initPolling(COMMAND_UPDATE_TIME_SECONDS);
+        } catch (InterruptedException e) {
+            Thread.currentThread().interrupt();
         }
-        clearState(true);
-        // reset the schedule for our next poll which at that time will reflect if our command was successful or not.
-        initPolling(COMMAND_UPDATE_TIME_SECONDS);
     }
 
     /**
@@ -349,7 +357,7 @@ public class AutelisHandler extends BaseThingHandler {
      * Poll the Autelis controller for updates. This will retrieve various xml documents and update channel states from
      * its contents.
      */
-    private void pollAutelisController() {
+    private void pollAutelisController() throws InterruptedException {
         logger.trace("Connecting to {}", baseURL);
 
         // clear our cached stated IF it is time.
@@ -466,16 +474,13 @@ public class AutelisHandler extends BaseThingHandler {
      * @param timeout
      * @return
      */
-    private synchronized String getUrl(String url, int timeout) {
+    private synchronized String getUrl(String url, int timeout) throws InterruptedException {
         // throttle commands for a very short time to avoid 'loosing' them
         long now = System.currentTimeMillis();
         long nextReq = lastRequestTime + THROTTLE_TIME_MILLISECONDS;
         if (nextReq > now) {
-            try {
+            logger.trace("Throttling request for {} mills", nextReq - now);
-                logger.trace("Throttling request for {} mills", nextReq - now);
+            Thread.sleep(nextReq - now);
-                Thread.sleep(nextReq - now);
-            } catch (InterruptedException ignored) {
-            }
         }
         String getURL = url + (url.contains("?") ? "&" : "?") + "timestamp=" + System.currentTimeMillis();
         logger.trace("Getting URL {} ", getURL);
@@ -490,7 +495,7 @@ public class AutelisHandler extends BaseThingHandler {
             }
             lastRequestTime = System.currentTimeMillis();
             return response.getContentAsString();
-        } catch (Exception e) {
+        } catch (ExecutionException | TimeoutException e) {
             logger.debug("Could not make http connection", e);
         }
         return null;

bundles/org.openhab.binding.automower/src/main/java/org/openhab/binding/automower/internal/rest/api/automowerconnect/AutomowerConnectApi.java

@@ -90,7 +90,10 @@ public class AutomowerConnectApi extends HusqvarnaApi {
         ContentResponse response;
         try {
             response = request.send();
-        } catch (InterruptedException | TimeoutException | ExecutionException e) {
+        } catch (TimeoutException | ExecutionException e) {
+            throw new AutomowerCommunicationException(e);
+        } catch (InterruptedException e) {
+            Thread.currentThread().interrupt();
             throw new AutomowerCommunicationException(e);
         }
         return response;

bundles/org.openhab.binding.avmfritz/src/main/java/org/openhab/binding/avmfritz/internal/hardware/FritzAhaWebInterface.java

@@ -242,9 +242,13 @@ public class FritzAhaWebInterface {
             String content = contentResponse.getContentAsString();
             logger.debug("GET response complete: {}", content);
             return content;
-        } catch (ExecutionException | InterruptedException | TimeoutException e) {
+        } catch (ExecutionException | TimeoutException e) {
             logger.debug("response failed: {}", e.getLocalizedMessage(), e);
             return null;
+        } catch (InterruptedException e) {
+            logger.debug("response interrupted: {}", e.getLocalizedMessage(), e);
+            Thread.currentThread().interrupt();
+            return null;
         }
     }
 

bundles/org.openhab.binding.boschshc/src/main/java/org/openhab/binding/boschshc/internal/devices/BoschSHCHandler.java

@@ -144,10 +144,15 @@ public abstract class BoschSHCHandler extends BaseThingHandler {
                 if (deviceService.affectedChannels.contains(channelUID.getIdWithoutGroup())) {
                     try {
                         deviceService.service.refreshState();
-                    } catch (InterruptedException | TimeoutException | ExecutionException | BoschSHCException e) {
+                    } catch (TimeoutException | ExecutionException | BoschSHCException e) {
                         this.updateStatus(ThingStatus.OFFLINE, ThingStatusDetail.OFFLINE.COMMUNICATION_ERROR,
                                 String.format("Error when trying to refresh state from service %s: %s",
                                         deviceService.service.getServiceName(), e.getMessage()));
+                    } catch (InterruptedException e) {
+                        this.updateStatus(ThingStatus.OFFLINE, ThingStatusDetail.OFFLINE.COMMUNICATION_ERROR,
+                                String.format("Interrupted refresh state from service %s: %s",
+                                        deviceService.service.getServiceName(), e.getMessage()));
+                        Thread.currentThread().interrupt();
                     }
                 }
             }
@@ -210,10 +215,15 @@ public abstract class BoschSHCHandler extends BaseThingHandler {
         try {
             BoschSHCBridgeHandler bridgeHandler = this.getBridgeHandler();
             return bridgeHandler.getState(deviceId, stateName, classOfT);
-        } catch (InterruptedException | TimeoutException | ExecutionException | BoschSHCException e) {
+        } catch (TimeoutException | ExecutionException | BoschSHCException e) {
             this.updateStatus(ThingStatus.OFFLINE, ThingStatusDetail.OFFLINE.COMMUNICATION_ERROR,
                     String.format("Error when trying to refresh state from service %s: %s", stateName, e.getMessage()));
             return null;
+        } catch (InterruptedException e) {
+            this.updateStatus(ThingStatus.OFFLINE, ThingStatusDetail.OFFLINE.COMMUNICATION_ERROR,
+                    String.format("Interrupted refresh state from service %s: %s", stateName, e.getMessage()));
+            Thread.currentThread().interrupt();
+            return null;
         }
     }
 
@@ -279,9 +289,13 @@ public abstract class BoschSHCHandler extends BaseThingHandler {
             TService service, TState state) {
         try {
             service.setState(state);
-        } catch (InterruptedException | TimeoutException | ExecutionException e) {
+        } catch (TimeoutException | ExecutionException e) {
             this.updateStatus(ThingStatus.OFFLINE, ThingStatusDetail.OFFLINE.COMMUNICATION_ERROR, String.format(
                     "Error when trying to update state for service %s: %s", service.getServiceName(), e.getMessage()));
+        } catch (InterruptedException e) {
+            this.updateStatus(ThingStatus.OFFLINE, ThingStatusDetail.OFFLINE.COMMUNICATION_ERROR, String
+                    .format("Interrupted update state for service %s: %s", service.getServiceName(), e.getMessage()));
+            Thread.currentThread().interrupt();
         }
     }
 

bundles/org.openhab.binding.boschshc/src/main/java/org/openhab/binding/boschshc/internal/devices/bridge/BoschSHCBridgeHandler.java

@@ -12,8 +12,7 @@
  */
 package org.openhab.binding.boschshc.internal.devices.bridge;
 
-import static org.eclipse.jetty.http.HttpMethod.GET;
+import static org.eclipse.jetty.http.HttpMethod.*;
-import static org.eclipse.jetty.http.HttpMethod.PUT;
 
 import java.lang.reflect.Type;
 import java.util.ArrayList;
@@ -30,7 +29,10 @@ import org.eclipse.jetty.client.api.Response;
 import org.eclipse.jetty.http.HttpStatus;
 import org.eclipse.jetty.util.ssl.SslContextFactory;
 import org.openhab.binding.boschshc.internal.devices.BoschSHCHandler;
-import org.openhab.binding.boschshc.internal.devices.bridge.dto.*;
+import org.openhab.binding.boschshc.internal.devices.bridge.dto.Device;
+import org.openhab.binding.boschshc.internal.devices.bridge.dto.DeviceStatusUpdate;
+import org.openhab.binding.boschshc.internal.devices.bridge.dto.LongPollResult;
+import org.openhab.binding.boschshc.internal.devices.bridge.dto.Room;
 import org.openhab.binding.boschshc.internal.exceptions.BoschSHCException;
 import org.openhab.binding.boschshc.internal.exceptions.LongPollingFailedException;
 import org.openhab.binding.boschshc.internal.exceptions.PairingFailedException;
@@ -233,6 +235,7 @@ public class BoschSHCBridgeHandler extends BaseBridgeHandler {
 
         } catch (InterruptedException e) {
             this.updateStatus(ThingStatus.UNKNOWN, ThingStatusDetail.UNKNOWN.NONE, "@text/offline.interrupted");
+            Thread.currentThread().interrupt();
         }
     }
 

bundles/org.openhab.binding.boschshc/src/main/java/org/openhab/binding/boschshc/internal/devices/bridge/LongPolling.java

@@ -116,10 +116,15 @@ public class LongPolling {
             logger.debug("Subscribe: Got subscription ID: {} {}", response.getResult(), response.getJsonrpc());
             String subscriptionId = response.getResult();
             return subscriptionId;
-        } catch (TimeoutException | ExecutionException | InterruptedException e) {
+        } catch (TimeoutException | ExecutionException e) {
             throw new LongPollingFailedException(
                     String.format("Error on subscribe (Http client: %s): %s", httpClient.toString(), e.getMessage()),
                     e);
+        } catch (InterruptedException e) {
+            Thread.currentThread().interrupt();
+            throw new LongPollingFailedException(
+                    String.format("Interrupted subscribe (Http client: %s): %s", httpClient.toString(), e.getMessage()),
+                    e);
         }
     }