Skip to content

Sending identification string before waiting for server's #506

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 1 commit into from

Conversation

martinprikryl
Copy link

According to RFC 4253 Section 4.2. Protocol Version Exchange:

When the connection has been established, both sides MUST send an identification string.

SSH.NET sends its identification string only after waiting for server's identification string. Amazon Managed SFTP server has the same issue. So you cannot connect to that SFTP server with SSH.NET. Both sides first wait for the other side to send the identification string before sending its own.

#505

According to RFC 4253 Section 4.2. Protocol Version Exchange:

> When the connection has been established, both sides MUST send an identification string.

SSH.NET sends its identification string only after waiting for server's identification string. Amazon Managed SFTP server has the same issue. So you cannot connect to that SFTP server with SSH.NET. Both sides first wait for the other side to send the identification string before sending its own.
@Strohbichler
Copy link

Hi Martin,
we implement this bugfix in our solution and it works.
THX :-)
Best Regards,
Strohbichler

@orudge
Copy link

orudge commented Nov 12, 2019

I came up with the same fix while debugging an issue while trying to connect to an SSH server via an HTTP proxy (direct connection and SOCKS5 connections worked, HTTP did not). If this could be merged it would be very helpful!

@drieseng
Copy link
Member

This is fixed in 2020.0.0-beta1.
Thanks!

@drieseng drieseng closed this Jun 28, 2020
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.

4 participants