Log and retry exceptions during the initial websocket connection instead of silently discarding them

Author: JulianVennen

CHANGELOG.md

@@ -1,3 +1,9 @@
+# 2.4.1
+## Fixes
+- Log and retry exceptions during the initial websocket connection instead of silently discarding them
+
+---
+
 # 2.4.0
 ## New Features
 - Allow proxying the new server management protocol added in 1.21.9 through the Exaroton API

src/main/java/com/exaroton/api/ws/WebSocketConnection.java

@@ -234,6 +234,11 @@ private void connect() {
                 .thenAccept(ws -> {
                     this.logger.debug("Connected to {}", uri);
                     this.client = ws;
+                })
+                .exceptionally(t -> {
+                    logger.error("Websocket connection failed", t);
+                    scheduleReconnect();
+                    return null;
                 });
     }
 
@@ -372,6 +377,15 @@ public CompletionStage<?> onClose(WebSocket webSocket, int statusCode, String re
             }
         }
 
+        this.scheduleReconnect();
+        return null;
+    }
+
+    /**
+     * This method is called when the connection is closed. It automatically starts a reconnect timer
+     * if autoReconnect is enabled.
+     */
+    private void scheduleReconnect() {
         if (this.shouldAutoReconnect()) {
             reconnectTimer = new Timer();
             logger.debug("Reconnecting in 5s");
@@ -382,7 +396,6 @@ public void run() {
                 }
             }, 5000, 5000);
         }
-        return null;
     }
 
     @Override