Skip to content

Add functionality to communicate with remote commands over stdin #1052

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

Closed
wants to merge 35 commits into from

Conversation

mdarocha
Copy link

This is a rebase and update of #144, with the sole purpose being adding functionality to communicate with commands executed over ssh using stdin.

This functionality is needed by ie. dotnet/Docker.DotNet#575

This PR reimplements functionality requested in #141, InputStream seems to have been lost in the original rewrite PR

drieseng and others added 30 commits October 15, 2017 11:18
* Create FUNDING.yml
* Add sponsor info.
# Conflicts:
#	build/sandcastle/SSH.NET.shfbproj
@WojciechNagorski
Copy link
Collaborator

@drieseng This is a really important issue for the entire .NET. We need this change for Docker.DotNet library that has 9.1M downloads. Currently, no one is able to create a real docker manager using .NET, because it can work only locally or with unsafe remote connections without encryption. Thanks to this PR the community will be able to communicate with Docker via the safe encrypted connection. Can you help with this PR?

@preetmyob
Copy link

@drieseng Would you have any idea if you're going to be able to have a look at this PR, please? Your help would really be appreciated. Thank you

@WojciechNagorski
Copy link
Collaborator

@mdarocha Tests do not pass. Can you check it?

@mdarocha
Copy link
Author

Unfortunately I don't have the time to work on this PR anymore. I'm closing it, if anybody wants to pick up this work, feel free to do it.

@mdarocha mdarocha closed this Jan 25, 2024
@WojciechNagorski
Copy link
Collaborator

This issue has been fixed in the 2024.0.0 version.

@WojciechNagorski WojciechNagorski added this to the 2024.0.0 milestone Feb 22, 2024
@WojciechNagorski
Copy link
Collaborator

Fixed by #1293

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.

5 participants