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

exec bash not working #174

Closed
ak-1 opened this issue Jul 18, 2022 · 3 comments
Closed

exec bash not working #174

ak-1 opened this issue Jul 18, 2022 · 3 comments
Labels
bug Something isn't working important An important issue that should be resolved quickly regression Regression bug

Comments

@ak-1
Copy link
Contributor

ak-1 commented Jul 18, 2022

Things like vopono exec bash do not work for me anymore since 7614548.
The command is executed but it exits again right away.
All REPL commands seem to be affected. GUI programs like firefox work though.
Looks like stdin is somehow not passed properly to the process.

@jamesmcm jamesmcm added bug Something isn't working important An important issue that should be resolved quickly regression Regression bug labels Jul 19, 2022
@jamesmcm
Copy link
Owner

I think it's because it takes over stdin - 7614548#diff-9c16fd9e2af528717067a09bcd97ff4982a30a74507ca4ef09704215b6c5f719R128

Could you test it removing line 129 from netns.rs ?

handle.stdin(Stdio::piped());

I'm on a work trip so I won't be able to re-release until Saturday at least.

We'll need to think of a better way of forwarding stdin for OpenConnect.

@ak-1
Copy link
Contributor Author

ak-1 commented Jul 20, 2022

Yes, after removing line 129 vopono exec bash is working again.

@jamesmcm jamesmcm mentioned this issue Jul 24, 2022
@jamesmcm
Copy link
Owner

Fixed in 0.10.1 release - #178

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working important An important issue that should be resolved quickly regression Regression bug
Projects
None yet
Development

No branches or pull requests

2 participants