Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

test: add test that we close stdin on SSH session close#18711

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
spikecurtis merged 1 commit intomainfromspike/18519-test-close-stdin
Jul 2, 2025

Conversation

spikecurtis
Copy link
Contributor

@spikecurtisspikecurtis commentedJul 2, 2025•
edited
Loading

closes#18519

Adds a unit test that verifies that we close the stdin to a non-TTY process when the SSH session connected to it exits.

c.f.#18519 (comment)

Validates that we match OpenSSH behavior.

@spikecurtisGraphite App
Copy link
ContributorAuthor

This stack of pull requests is managed byGraphite. Learn more aboutstacking.

@spikecurtisspikecurtis requested review fromjohnstcn andmafredri and removed request forjohnstcnJuly 2, 2025 12:02
@spikecurtisspikecurtis marked this pull request as ready for reviewJuly 2, 2025 12:03
@spikecurtisspikecurtisforce-pushed thespike/18519-test-close-stdin branch from8f0098e to9c14eb8CompareJuly 2, 2025 12:04
@spikecurtisspikecurtisforce-pushed thespike/18519-test-close-stdin branch from9c14eb8 to2184b16CompareJuly 2, 2025 12:09
@spikecurtisspikecurtis merged commit59c8b56 intomainJul 2, 2025
30 checks passed
@spikecurtisGraphite App
Copy link
ContributorAuthor

Merge activity

@spikecurtisspikecurtis deleted the spike/18519-test-close-stdin branchJuly 2, 2025 12:23
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsJul 2, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers

@mafredrimafredrimafredri approved these changes

@johnstcnjohnstcnjohnstcn approved these changes

Assignees

@spikecurtisspikecurtis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

VS Code Desktop connection drops shortly after reconnecting
3 participants
@spikecurtis@mafredri@johnstcn
[8]ページ先頭
©2009-2025 Movatter.jp