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

"known_hosts" records incorrect SFTP address for servers with non-22 ports #10772

Closed
cyberduck opened this issue Aug 3, 2019 · 2 comments
Closed
Assignees
Labels
bug fixed sftp SFTP Protocol Implementation
Milestone

Comments

@cyberduck
Copy link
Collaborator

c8fd01b created the issue

Hi,

In Cyberduck 7.0.2, the ~/.ssh/known_hosts file will not record SFTP server's port number, invalids the "Remember Server Signature" function. Here is the procedure to reproduce:

  1. Connect to an SFTP server that has a non-standard port, let's say "ssh.example.com:54321".
  2. Cyberduck asks you whether to remember this server signature. Choose "Always".
  3. Find a new line in the ~/.ssh/known_hosts file:
ssh.example.com ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIJRw0CtHEseIIU/6OUf/NYNKXPs5tCu6EfwUty3bzZcm
  1. Disconnect the server and repeat Step 1. Now Cyberduck asks you again for saving the signature. Choose "Always" again.
  2. Now there are two duplicate lines in ~/.ssh/known_hosts file:
ssh.example.com ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIJRw0CtHEseIIU/6OUf/NYNKXPs5tCu6EfwUty3bzZcm
ssh.example.com ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIJRw0CtHEseIIU/6OUf/NYNKXPs5tCu6EfwUty3bzZcm

This bug does not exist in Cyberduck 7.0.1. In this version, the following line is added to ~/.ssh/known_hosts when you choose to remember the signature:

[ssh.example.com]:54321 ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIJRw0CtHEseIIU/6OUf/NYNKXPs5tCu6EfwUty3bzZcm

which is correct. Cyberduck will not ask you to remember it again during the second connection.

You can see that the missing PORT number in Cyberduck 7.0.2 causes this bug.

@cyberduck
Copy link
Collaborator Author

@dkocher commented

In 648b967.

@cyberduck
Copy link
Collaborator Author

@dkocher commented

#10796 closed as duplicate.

@iterate-ch iterate-ch locked as resolved and limited conversation to collaborators Nov 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug fixed sftp SFTP Protocol Implementation
Projects
None yet
Development

No branches or pull requests

2 participants