From 7bb7b9b131fb95e3be67b3925402d5f1103f7378 Mon Sep 17 00:00:00 2001
From: griffi-gh <prasol258@gmail.com>
Date: Mon, 13 Feb 2023 02:30:33 +0100
Subject: [PATCH] kick inactive clients

---
 kubi-udp/src/server.rs | 25 ++++++++++++++++++++++---
 1 file changed, 22 insertions(+), 3 deletions(-)

diff --git a/kubi-udp/src/server.rs b/kubi-udp/src/server.rs
index 18fa508..daf4b9a 100644
--- a/kubi-udp/src/server.rs
+++ b/kubi-udp/src/server.rs
@@ -74,12 +74,17 @@ impl<S, R> Server<S, R> where S: Message, R: Message {
     })
   }
 
-  fn send_to_addr(&self, addr: SocketAddr, packet: IdServerPacket<S>) -> Result<()> {
+
+  fn send_to_addr_inner(socket: &UdpSocket, addr: SocketAddr, packet: IdServerPacket<S>) -> Result<()> {
     let bytes = bincode::encode_to_vec(packet, BINCODE_CONFIG)?;
-    self.socket.send_to(&bytes, addr)?;
+    socket.send_to(&bytes, addr)?;
     Ok(())
   }
 
+  fn send_to_addr(&self, addr: SocketAddr, packet: IdServerPacket<S>) -> Result<()> {
+    Self::send_to_addr_inner(&self.socket, addr, packet)
+  }
+
   fn send_packet(&self, packet: IdServerPacket<S>) -> Result<()> {
     let Some(id) = packet.0 else {
       bail!("send_to_client call without id")
@@ -144,7 +149,21 @@ impl<S, R> Server<S, R> where S: Message, R: Message {
   }
   
   pub fn update(&mut self) -> Result<()> {
-    //TODO client timeout
+    //kick inactive clients
+    self.clients.retain(|&id, client| {
+      if client.timeout.elapsed() > self.config.client_timeout {
+        if let Err(_) = Self::send_to_addr_inner(&self.socket, client.addr, IdServerPacket(
+          Some(id), ServerPacket::Disconnected("Timed out".into())
+        )) {
+          log::warn!("Client {id} timed out and we failed to send the kick packet. This shouldn't reaally matter")
+        } else {
+          log::info!("Client {id} timed out");
+        }
+        return false
+      }
+      true
+    });
+
     let mut buf = [0; u16::MAX as usize];
     loop {
       match self.socket.recv_from(&mut buf) {