Skip to content

refactor: PTY & SSH #7100

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.

Sign up for GitHub

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

Merged
merged 22 commits into from
Apr 24, 2023
Merged

refactor: PTY & SSH #7100

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
0075d7d
Add ssh tests for longoutput, orphan
spikecurtis Apr 11, 2023
a491d4f
PTY/SSH tests & improvements
spikecurtis Apr 12, 2023
2cf357a
Fix some tests
spikecurtis Apr 12, 2023
28c0646
Fix linting
spikecurtis Apr 12, 2023
872e357
fmt
spikecurtis Apr 12, 2023
3f21e30
Fix windows test
spikecurtis Apr 13, 2023
e83ff6e
Windows copy test
spikecurtis Apr 14, 2023
b610579
WIP Windows pty handling
spikecurtis Apr 14, 2023
90bfe94
Fix truncation tests
spikecurtis Apr 18, 2023
d6e131c
Appease linter/fmt
spikecurtis Apr 18, 2023
2c9c6ef
Fix typo
spikecurtis Apr 18, 2023
e39e885
Rework truncation test to not assume OS buffers
spikecurtis Apr 18, 2023
8ec3d1f
Disable orphan test on Windows --- uses sh
spikecurtis Apr 18, 2023
df424e6
agent_test running SSH in pty use ptytest.Start
spikecurtis Apr 19, 2023
50e3fec
More detail about closing pseudoconsole on windows
spikecurtis Apr 19, 2023
c09083e
Code review fixes
spikecurtis Apr 20, 2023
439107d
Rearrange ptytest method order
spikecurtis Apr 20, 2023
c6a3229
Protect pty.Resize on windows from races
spikecurtis Apr 21, 2023
aa94546
Fix windows bugs
spikecurtis Apr 21, 2023
0f07cb9
PTY doesn't extend PTYCmd
spikecurtis Apr 24, 2023
eaabb3a
merge branch 'main' of https://github.com/coder/coder into spike/pty-ssh
spikecurtis Apr 24, 2023
50060d8
Fix windows types
spikecurtis Apr 24, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix typo 
Signed-off-by: Spike Curtis <spike@coder.com>
  • Loading branch information
@spikecurtis
spikecurtis committed Apr 18, 2023
commit 2c9c6efbc10dc2cac4a508518b0c3597e200387d
2 changes: 1 addition & 1 deletion pty/pty_windows.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ func (p *windowsProcess) waitInternal() {
if ret < 0 && p.cmdErr == nil {
// if we already have an error from the command, prefer that error
// but if the command succeeded and closing the PseudoConsole fails
// then record that errror so that we have a chance to see it
// then record that error so that we have a chance to see it
p.cmdErr = err
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It'd be nice to log the error whether it's kept or discarded

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice, but I don't have a logger, and I'd have to change all the init signatures to get one, so I decided on this.

Another alternative I considered is panicking---something pretty messed up at the OS level is happening if we get an error here. We're pretty much guaranteed to crash the agent if we panic, so it's likely we'd get told about it.

}
p.pw.console = windows.InvalidHandle
Expand Down