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

add Session::channel_direct_streamlocal #291

Merged

Conversation

demosdemon
Copy link
Contributor

@demosdemon demosdemon commented Jun 15, 2023

Support for this extension was officially added to libssh2 in v1.11.0 via libssh2/libssh2#945.

This PR updates libssh2 to this new version and adds the bindings for libssh2_channel_direct_streamlocal_ex.

Closes #279

See libssh2/libssh2@4f0f4bf for reference to explain the build.rs changes.

Test Plan:

https://gist.github.com/demosdemon/475db259ebf63cfb35cb1fb4186a67cd

@demosdemon demosdemon marked this pull request as ready for review June 15, 2023 23:03
@demosdemon demosdemon force-pushed the pr/channel_direct_streamlocal branch from b18f3fc to 9b66bce Compare June 15, 2023 23:50
@demosdemon demosdemon force-pushed the pr/channel_direct_streamlocal branch from 9b66bce to dc441f9 Compare June 16, 2023 00:10
@twitzelbos
Copy link
Contributor

twitzelbos commented Sep 20, 2023

This is great! Is there anything blocking this from being merged? I guess the lack of maintainers in this crate?

@yodaldevoid yodaldevoid merged commit f097dbd into alexcrichton:master Sep 21, 2023
@yodaldevoid
Copy link
Collaborator

yodaldevoid commented Sep 21, 2023

Thank you for the new feature and the very detailed PR!

@twitzelbos
Copy link
Contributor

Actually messing with it in my fork, I found the test breaks the windows build. I can make a small PR for that.

@yodaldevoid
Copy link
Collaborator

Actually messing with it in my fork, I found the test breaks the windows build. I can make a small PR for that.

I also noticed that after I had merged this and had to step away yesterday. If you want to make a quick PR I would appreciate it.

@twitzelbos
Copy link
Contributor

Done, PR #302

@demosdemon demosdemon deleted the pr/channel_direct_streamlocal branch September 22, 2023 19:43
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.

Unix Socket Tunnel?
3 participants