Merge pull request #84 from tryptichon/master

Version 2.2.1
arago · Jul 17, 2020 · f838f81 · f838f81
2 parents e068242 + 8c1502c
commit f838f81
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 12 deletions.
diff --git a/VERSIONS.md b/VERSIONS.md
@@ -4,4 +4,4 @@
 | --------------- | ------ | ----------------------- |
 | 1.6.x | 1.6 | 6.1.x |
 | 2.0.x | master | 7.0.x |
-| 2.1.x | master | 7.1.x |
+| 2.2.x | master | 7.1.x |
diff --git a/java/hiro-action-client/pom.xml b/java/hiro-action-client/pom.xml
@@ -4,11 +4,11 @@
   <groupId>co.arago</groupId>
   <artifactId>hiro-action-client</artifactId>
   <name>${project.artifactId}</name>
-  <version>2.2.0</version>
+  <version>2.2.1</version>
   <parent>
     <groupId>co.arago</groupId>
     <artifactId>hiro-client-all</artifactId>
-    <version>2.2.0</version>
+    <version>2.2.1</version>
   </parent>
 
   <properties>

diff --git a/java/hiro-client/pom.xml b/java/hiro-client/pom.xml
@@ -4,11 +4,11 @@
   <groupId>co.arago</groupId>
   <artifactId>hiro-client</artifactId>
   <name>${project.artifactId}</name>
-  <version>2.2.0</version>
+  <version>2.2.1</version>
   <parent>
     <groupId>co.arago</groupId>
     <artifactId>hiro-client-all</artifactId>
-    <version>2.2.0</version>
+    <version>2.2.1</version>
   </parent>
 
   <properties>

diff --git a/java/hiro-client/src/main/java/co/arago/hiro/client/rest/DefaultWebSocketClient.java b/java/hiro-client/src/main/java/co/arago/hiro/client/rest/DefaultWebSocketClient.java
@@ -222,9 +222,7 @@ private void connect(boolean isReconnecting) {
             LOG.log(Level.FINEST, "connecting " + this);
         }
 
-        WebSocketUpgradeHandler.Builder upgradeHandlerBuilder = new WebSocketUpgradeHandler.Builder();
-
-        WebSocketUpgradeHandler wsHandler = upgradeHandlerBuilder
+        WebSocketUpgradeHandler wsHandler = new WebSocketUpgradeHandler.Builder()
                 .addWebSocketListener(new DefaultWebSocketListener(isReconnecting)).build();
 
         try {
@@ -265,19 +263,24 @@ private void connect(boolean isReconnecting) {
      * InterruptedException while sleeping.
      */
     private synchronized void reconnect() {
+        if (!running) {
+            return;
+        }
+
         Duration nextTryDelay = Duration.ZERO;
 
         closeWs();
 
         while (true) {
             try {
+                long delay = nextTryDelay.getSeconds() * 1000;
+                if (delay > 0)
+                    wait(delay);
+
                 if (!running) {
                     return;
                 }
 
-                long delay = nextTryDelay.getSeconds() * 1000;
-                Thread.sleep(delay);
-
                 if (LOG.isLoggable(Level.INFO)) {
                     LOG.log(Level.INFO, "Reconnecting " + this);
                 }
@@ -426,6 +429,7 @@ public synchronized long sendMessage(String type, Map<String, String> headers, M
     public synchronized void close() {
         running = false;
 
+        notifyAll();
         closeWs();
 
         executor.shutdownNow();

diff --git a/java/pom.xml b/java/pom.xml
@@ -7,7 +7,7 @@
     <artifactId>hiro-client-all</artifactId>
     <name>${project.artifactId}</name>
     <packaging>pom</packaging>
-    <version>2.2.0</version>
+    <version>2.2.1</version>
 
     <properties>
         <maven.exec.skip>false</maven.exec.skip>