gowalk-cicd 1.0.83 → 1.0.84

package/action/.daemux-version

@@ -1 +1 @@
-1.0.83
+1.0.84

package/android-action/.daemux-version

@@ -1 +1 @@
-1.0.83
+1.0.84

package/android-action/scripts/jvm_proxy_relay.py

@@ -39,11 +39,14 @@ class Relay:
             self.ready.set()
             if self.server:
                 self.server.close()
-                self.loop.run_until_complete(self.server.wait_closed())
+            # Python 3.12+ waits for active clients in Server.wait_closed().
+            # Their handlers own the connections, so cancel and drain them first.
             tasks = asyncio.all_tasks(self.loop)
             for task in tasks:
                 task.cancel()
             self.loop.run_until_complete(asyncio.gather(*tasks, return_exceptions=True))
+            if self.server:
+                self.loop.run_until_complete(self.server.wait_closed())
             self.loop.close()
 
     async def _upstream(self, request: bytes):

package/android-action/scripts/test_jvm_proxy_cleanup.py

@@ -0,0 +1,44 @@
+"""Open client sockets must not keep the command's owned relay alive on Python 3.12+."""
+import asyncio
+import socket
+import unittest
+
+import jvm_proxy_relay as relay
+from test_jvm_proxy_relay import upstream
+
+
+async def accepted():
+    while not any(task.get_coro().__name__ == "_handle" for task in asyncio.all_tasks()):
+        await asyncio.sleep(0.001)
+
+
+class RelayCleanupTests(unittest.TestCase):
+    def test_shutdown_cancels_open_clients_before_waiting_for_server_connections(self):
+        with upstream() as (proxy, _seen):
+            for connected in (False, True):
+                with self.subTest(established_tunnel=connected):
+                    owned = relay.Relay(proxy)
+                    owned.thread.start()
+                    self.assertTrue(owned.ready.wait(5))
+                    client = socket.create_connection(("127.0.0.1", owned.port), timeout=5)
+                    try:
+                        if connected:
+                            client.sendall(b"CONNECT unresolvable.invalid:443 HTTP/1.1\r\nHost: ignored\r\n\r\n")
+                            self.assertIn(b"200", client.recv(4096))
+                        else:
+                            # Synchronize on acceptance while deliberately leaving the request unfinished.
+                            asyncio.run_coroutine_threadsafe(accepted(), owned.loop).result(timeout=5)
+                        owned.loop.call_soon_threadsafe(owned.loop.stop)
+                        owned.thread.join(timeout=2)
+                        self.assertFalse(owned.thread.is_alive(), "relay cleanup waits on a client it must close")
+                        self.assertTrue(owned.loop.is_closed())
+                        self.assertEqual(client.recv(4096), b"")
+                        with self.assertRaises(OSError):
+                            socket.create_connection(("127.0.0.1", owned.port), timeout=0.2)
+                    finally:
+                        client.close()
+                        owned.thread.join(timeout=5)
+
+
+if __name__ == "__main__":
+    unittest.main()

package/backend-action/.daemux-version

@@ -1 +1 @@
-1.0.83
+1.0.84

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gowalk-cicd",
-  "version": "1.0.83",
+  "version": "1.0.84",
   "description": "Zero-config GitHub Actions delivery for iOS TestFlight and Android Google Play.",
   "type": "module",
   "bin": {