fix(tcp): use threads to allow multiple conns

komorebi/src/process_command.rs

@@ -114,10 +114,7 @@ pub fn listen_for_commands_tcp(wm: Arc<Mutex<WindowManager>>, port: usize) {
 
                     tracing::info!("listening for incoming tcp messages from {}", &addr);
 
-                    match read_commands_tcp(&wm, &mut stream, &addr) {
-                        Ok(()) => {}
-                        Err(error) => tracing::error!("{}", error),
-                    }
+                    read_commands_tcp(&wm, &mut stream, &addr);
                 }
                 Err(error) => {
                     tracing::error!("{}", error);
@@ -939,20 +936,19 @@ pub fn read_commands_uds(wm: &Arc<Mutex<WindowManager>>, stream: UnixStream) ->
     Ok(())
 }
 
-pub fn read_commands_tcp(
-    wm: &Arc<Mutex<WindowManager>>,
-    stream: &mut TcpStream,
-    addr: &str,
-) -> Result<()> {
+pub fn read_commands_tcp(wm: &Arc<Mutex<WindowManager>>, stream: &mut TcpStream, addr: &str) {
+    let addr = addr.to_string();
+    let wm = wm.clone();
+    let stream = stream.try_clone().unwrap();
+    std::thread::spawn(move || -> Result<()> {
         let mut stream = BufReader::new(stream);
-
         loop {
             let mut buf = vec![0; 1024];
             match stream.read(&mut buf) {
                 Err(..) => {
                     tracing::warn!("removing disconnected tcp client: {addr}");
                     let mut connections = TCP_CONNECTIONS.lock();
-                connections.remove(addr);
+                    connections.remove(&addr);
                     break;
                 }
                 Ok(size) => {
@@ -963,7 +959,7 @@ pub fn read_commands_tcp(
                     } else {
                         tracing::warn!("client sent an invalid message, disconnecting: {addr}");
                         let mut connections = TCP_CONNECTIONS.lock();
-                    connections.remove(addr);
+                        connections.remove(&addr);
                         break;
                     };
 
@@ -971,9 +967,9 @@ pub fn read_commands_tcp(
 
                     if wm.is_paused {
                         return match message {
-                        SocketMessage::TogglePause | SocketMessage::State | SocketMessage::Stop => {
-                            Ok(wm.process_command(message)?)
-                        }
+                            SocketMessage::TogglePause
+                            | SocketMessage::State
+                            | SocketMessage::Stop => Ok(wm.process_command(message)?),
                             _ => {
                                 tracing::trace!("ignoring while paused");
                                 Ok(())
@@ -991,4 +987,5 @@ pub fn read_commands_tcp(
         }
 
         Ok(())
+    });
 }