Skip to content

[v16] Refactor tracessh.Client to cover untraced paths - session requests#59789

Merged
Joerger merged 3 commits intobranch/v16from
joerger/v16/ssh-session-client
Oct 6, 2025
Merged

[v16] Refactor tracessh.Client to cover untraced paths - session requests#59789
Joerger merged 3 commits intobranch/v16from
joerger/v16/ssh-session-client

Conversation

@Joerger
Copy link
Copy Markdown
Contributor

@Joerger Joerger commented Sep 30, 2025

Backport #59291 to branch/v16

…#59291)

* Move ssh session helper into a separate file.

* Replace ssh session ChannelRequestCallback with a more x/crypto/ssh-like approach to request channel handling.

* Update comments.

* Move new request handling logic into tracessh client.

* Fix goroutine leaks with refactors.

* Add test.

* Cleanup.

* Rename methods.

* Fix lint.

* Address comments; Fix error and cleanup handling issues in tracing ssh tests.

* Increase test timeouts.

* Fix test data race.
@Joerger Joerger added backport no-changelog Indicates that a PR does not require a changelog entry labels Sep 30, 2025
@Joerger Joerger added this pull request to the merge queue Oct 6, 2025
Merged via the queue into branch/v16 with commit 5a132d4 Oct 6, 2025
40 of 41 checks passed
@Joerger Joerger deleted the joerger/v16/ssh-session-client branch October 6, 2025 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport no-changelog Indicates that a PR does not require a changelog entry size/md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants