In case of a successful DDoS, prevent wasting resources evaluating blacklist

and other stuff as the server is full. This is done this way, not to save
the game server from the DDoS, but to save the remainder of the VM.

1 files changed, 9 insertions, 9 deletions

diff --git a/server.py b/server.py
index 2428183..2dee2ba 100755
--- a/server.py
+++ b/server.py
@@ -82,6 +82,15 @@ class WebSocketConn(WebSocket):
         #    client.send_message(self.address[0] + u' - connected')
         stdout(self.address[0] + u' - connected')
 
+        # Keep only a "sane" amount of clients connected to the server.
+        # This program must not, under any circumstances, interfer
+        # on the operation of Moubootaur Legends, which has total priority
+        # over the VM resources
+        if (len(clients) > MAX_CLIENTS):
+            self.close(self, status=1000, reason='Server is full')
+        else:
+            clients.append(self)
+
         # Blacklisted
         if (security.is_banned(self.address[0])):
             stdout("Found in K-Line, connection was dropped.")
@@ -92,15 +101,6 @@ class WebSocketConn(WebSocket):
         # TODO: Either auto-ban or limit to <= 3 connections from same IP at once
         # TODO: Also, inheir the previous connection MS score and MS Auth
 
-        # By last, keep a "sane" amount of clients connected
-        # This program must not, under any circumstances, interfer
-        # on the operation of Moubootaur Legends, which has total priority
-        # over the VM resources
-        if (len(clients) > MAX_CLIENTS):
-            self.close(self, status=1000, reason='Server is full')
-        else:
-            clients.append(self)
-
         # Extend self class
         self.MS_score = 0
         self.MS_auth  = False