cache: don't complete haneshake future until after state is changed to connected

2017-07-18 20:48:05 -04:00
parent 45e41e06d6
commit 5a270cfc6d
1 changed files with 3 additions and 2 deletions
--- a/cache/src/main/java/net/runelite/cache/downloader/CacheClient.java
+++ b/cache/src/main/java/net/runelite/cache/downloader/CacheClient.java
@@ -134,10 +134,9 @@ public class CacheClient implements AutoCloseable
 	{
 		assert handshakeFuture != null;

-		handshakeFuture.complete(response);
-
 		if (response != HelloHandshake.RESPONSE_OK)
 		{
+			handshakeFuture.complete(response);
 			close();
 			return;
 		}
@@ -153,6 +152,8 @@ public class CacheClient implements AutoCloseable
 		state = ClientState.CONNECTED;

 		logger.info("Client is now connected!");
+
+		handshakeFuture.complete(response);
 	}

 	@Override