refactor(connector.py): move time.sleep to the top of Connector._loop() to avoid infinite immediate retrying if data empty

This commit is contained in:
2025-08-28 20:51:35 +02:00
parent 18e60ee8c7
commit 43e61e7e68

View File

@@ -83,6 +83,7 @@ class Connector:
self.socket.connect((self.host, self.port)) self.socket.connect((self.host, self.port))
self.socket.settimeout(self.socket_timeout) self.socket.settimeout(self.socket_timeout)
self.logger.info(f"[+] Connected to {self.host}:{self.port}") self.logger.info(f"[+] Connected to {self.host}:{self.port}")
self.send_hello()
except ( except (
socket.timeout, socket.timeout,
ConnectionRefusedError, ConnectionRefusedError,
@@ -131,6 +132,7 @@ class Connector:
def _loop(self) -> None: def _loop(self) -> None:
self.logger.debug("Starting main loop...") self.logger.debug("Starting main loop...")
while True: while True:
time.sleep(1)
data: bytes = self.receive() data: bytes = self.receive()
if not data: if not data:
continue continue
@@ -140,7 +142,6 @@ class Connector:
# self.logger.debug("[.] ACK verified") # self.logger.debug("[.] ACK verified")
# else: # else:
# self.logger.error("[!] ACK verification failed") # self.logger.error("[!] ACK verification failed")
time.sleep(1)
def run(self) -> None: def run(self) -> None:
self.logger.debug("Running Connector...") self.logger.debug("Running Connector...")