fix: release socket table lock before blocking connect and improve timeout test

Signed-off-by: danbugs <danilochiarlone@gmail.com>

Author: danbugs

host/src/lib.rs

@@ -1201,8 +1201,10 @@ fn register_net_tools(
         let addr = parse_sockaddr(&args)?;
         pol.check(&addr)?;
         let sa: SockAddr = addr.into();
-        let tbl = t.lock().unwrap();
-        let sock = tbl.get_socket(fd)?;
+        let sock = {
+            let tbl = t.lock().unwrap();
+            tbl.get_socket(fd)?.try_clone()?
+        };
         sock.connect_timeout(&sa, SOCKET_TIMEOUT)?;
         Ok(json!({}))
     });
@@ -3224,18 +3226,23 @@ mod tests {
     fn net_socket_has_default_timeout() {
         let mut tools = ToolRegistry::new();
         let exit_code = Arc::new(AtomicI32::new(0));
-        register_internal_tools(&mut tools, &exit_code, Some(&NetworkPolicy::AllowAll), None);
+        let socket_table =
+            register_internal_tools(&mut tools, &exit_code, Some(&NetworkPolicy::AllowAll), None);
 
         let req = br#"{"name":"net_socket","args":{"family":2,"type":1}}"#;
         let resp = tools.dispatch(req);
         let s = std::str::from_utf8(&resp).unwrap();
+        assert!(!s.contains("error"), "socket creation should succeed: {s}");
+
         let v: serde_json::Value = serde_json::from_str(s).unwrap();
         let fd = v["result"]["fd"].as_u64().unwrap();
-
-        // Verify the socket was created successfully
         assert!(fd > 0);
-        // The timeout is set internally; we verify it works by checking
-        // that the response is valid (no error from set_read_timeout)
-        assert!(!s.contains("error"), "socket creation should succeed: {s}");
+
+        // Verify timeouts are actually set to SOCKET_TIMEOUT (30s)
+        let tbl = socket_table.unwrap();
+        let tbl = tbl.lock().unwrap();
+        let sock = tbl.get_socket(fd).unwrap();
+        assert_eq!(sock.read_timeout().unwrap(), Some(SOCKET_TIMEOUT));
+        assert_eq!(sock.write_timeout().unwrap(), Some(SOCKET_TIMEOUT));
     }
 }