chef-cookbooks / ssh_known_hosts

Development repository for Chef Cookbook ssh_known_hosts
https://supermarket.chef.io/cookbooks/ssh_known_hosts
Apache License 2.0
72 stars 77 forks source link

do not write port number if it is 22 #71

Closed axos88 closed 7 years ago

axos88 commented 7 years ago

Generated ssh_known_hosts was not working for hosts on the default port. (Maybe syntax for specifying port is broken?)

Before merging, please verify that removing the port specification does not add the key as accepted for all ports, which may be undesirable.

Obvious fix.

tas50 commented 7 years ago

It looks like the syntax introduced in https://github.com/chef-cookbooks/ssh_known_hosts/pull/54 is wrong. I'm working on a fix and some tests so ensure we write out the correct format.