OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

fix websocket leaks

Open rbren opened this issue 1 year ago • 0 comments

End-user friendly description of the problem this fixes or functionality that this introduces

  • [ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below no changelog

Give a summary of what the PR does, explaining any non-trivial design decisions

Pretty sure we're leaking FDs thanks to not closing websockets!


Link of any specific issues this addresses


To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:277d9b2-nikolaik   --name openhands-app-277d9b2   docker.all-hands.dev/all-hands-ai/openhands:277d9b2

rbren avatar Nov 13 '24 22:11 rbren