Fix duplicate shutdown messages for clients

qixils · Nov 28, 2021 · afbf44a · afbf44a
1 parent 332ceda
commit afbf44a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 14 deletions.
diff --git a/src/main/java/dev/qixils/crowdcontrol/socket/DummyResponse.java b/src/main/java/dev/qixils/crowdcontrol/socket/DummyResponse.java
@@ -11,7 +11,7 @@
 import java.nio.charset.StandardCharsets;
 import java.util.Objects;
 
-class DummyResponse implements JsonObject {
+final class DummyResponse implements JsonObject {
 	int id = 0;
 	ResultType status;
 	String message;

diff --git a/src/main/java/dev/qixils/crowdcontrol/socket/ServerSocketManager.java b/src/main/java/dev/qixils/crowdcontrol/socket/ServerSocketManager.java
@@ -65,21 +65,8 @@ private void loop() {
         }
     }
 
-    private void writeResponse(@NotNull DummyResponse response) {
-        for (SocketThread socketThread : socketThreads) {
-            if (socketThread.isSocketActive()) {
-                response.write(socketThread.socket);
-            }
-        }
-    }
-
     @Override
     public void shutdown(@Nullable Request cause, @Nullable String reason) throws IOException {
-        writeResponse(DummyResponse.from(cause, reason));
-        rawShutdown(cause, reason);
-    }
-
-    private void rawShutdown(@Nullable Request cause, @Nullable String reason) throws IOException {
         running = false;
         for (SocketThread socketThread : socketThreads) {
             if (socketThread.isSocketActive())