Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wrapped command not killed/closed in certain conditions #125

Open
jd-jedi opened this issue Apr 17, 2014 · 0 comments
Open

wrapped command not killed/closed in certain conditions #125

jd-jedi opened this issue Apr 17, 2014 · 0 comments
Labels
bug Something isn't working feature New feature or request patchwelcome

Comments

@jd-jedi
Copy link

jd-jedi commented Apr 17, 2014

Hi guys,

Trying to use wrap option. Have socat proxying for actual vnc server.

./utils/websockify --timeout 20 --run-once --ssl-only 6900 -- socat... params.

I am expecting that when the websockify exits because of timeout, it closes/kills the socat as well. btw, it does work, if I press crt-c.

I guess, we should add do_cleanup() and use it to clean the self.cmd in some of these conditions. I tried putting it in, but it is getting called twice and dont know correct way of closing popen handle. May be has to do with multiprocess? (may need to spend more time on this). Would appreciate if some who understands the code can fix the issue/help fix it.

Thanks
/Jd

@DirectXMan12 DirectXMan12 added feature New feature or request bug Something isn't working patchwelcome labels Sep 23, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature New feature or request patchwelcome
Projects
None yet
Development

No branches or pull requests

2 participants