OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

fix(sandbox): SSH box `.execute` misalign issue

Open xingyaoww opened this issue 1 year ago • 2 comments

In DockerSSHBox, sometimes the previous execution observation will come out in subsequent ssh.before.decode, which causes (1) exit code reading sometime to fail, and/or (2) output of a previous command to come out in a subsequent command. This PR tries to fix it.

xingyaoww avatar Apr 23 '24 10:04 xingyaoww

@rbren pxssh suppose to handles it all for us until I realized the issue 😅

xingyaoww avatar Apr 23 '24 13:04 xingyaoww

Should we maybe open a bug against pexpect?

rbren avatar Apr 23 '24 21:04 rbren

Good idea! I will start an issue.

xingyaoww avatar Apr 24 '24 04:04 xingyaoww