markolson / chef-ssh

Chef cookbook for managing some mildly-difficult-to-automate SSH configuration
39 stars 54 forks source link

resource.name is used for Host instead of resource.host #39

Closed 5c077yP closed 9 years ago

5c077yP commented 9 years ago

hey,

i'm working with version 0.10.2 and for the lwrp ssh_config it seems to use the name attribute for the Host instead of using the host attribute.

example:

ssh_config 'github config for user root' do
  host 'github.com'
  options(
    'HostName' => 'github.com',
    'User' => 'git',
    'IdentityFile' => '/root/.ssh/id_rsa_github'
  )
end

creates the following:

$ cat /root/.ssh/config
# Created by Chef for tm-infrastructure-chef

Host github config for user root
  HostName github.com
  User git
  IdentityFile /root/.ssh/id_rsa_github
tejaycar commented 9 years ago

Yes, it sure does. I'll get a fix out as quickly as I can. Thanks for the catch

5c077yP commented 9 years ago

wow, thanks for the quick response

tejaycar commented 9 years ago

Alright, it's released and on supermarket. Thanks again for spotting it. Should have caught that in my tests, but I didn't have coverage.

5c077yP commented 9 years ago

nice job, works perfectly!