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

Wire up agent forward for libssh backend #5345

Merged
merged 2 commits into from
May 8, 2024
Merged

Conversation

Riatre
Copy link
Contributor

@Riatre Riatre commented Apr 27, 2024

TSIA.

Depends on wez/libssh-rs#24

There's a drive-by change in sftpwrap.rs for bumping to new libssh-rs.

@Riatre Riatre force-pushed the agent-forward-2 branch 3 times, most recently from a85d1fd to 4ff5a9a Compare May 8, 2024 11:54
@Riatre Riatre changed the title wip: wire up agent forward for libssh-rs Wire up agent forward for libssh backend May 8, 2024
@Riatre Riatre marked this pull request as ready for review May 8, 2024 11:55
Copy link
Owner

@wez wez left a comment

Choose a reason for hiding this comment

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

I think this looks good; let's update to point at crates.io and I think we're good!
Thank you!

wezterm-ssh/src/sessioninner.rs Show resolved Hide resolved
wezterm-ssh/Cargo.toml Outdated Show resolved Hide resolved
TSIA.

There's a drive-by change in sftpwrap.rs for bumping to new libssh-rs.
@wez wez merged commit 9b811c7 into wez:main May 8, 2024
16 of 17 checks passed
wez added a commit that referenced this pull request May 8, 2024
@wez
Copy link
Owner

wez commented May 8, 2024

Thanks!

@Riatre Riatre deleted the agent-forward-2 branch May 8, 2024 16:59
saep pushed a commit to saep/wezterm that referenced this pull request Jul 14, 2024
* Wire up agent forward for libssh backend

TSIA.

There's a drive-by change in sftpwrap.rs for bumping to new libssh-rs.
saep pushed a commit to saep/wezterm that referenced this pull request Jul 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants