Merge pull request #40 from tejaycar/master

fix gitlab 39.  Use config.host rather than config.name in the config LW...

Author: tejaycar

CHANGELOG.md

@@ -1,4 +1,7 @@
 # CHANGELOG for ssh
+## 0.10.4
+* fix github #39 where we use the resource name rather than host (The name attribute) in the config LWRP
+
 ## 0.10.2
 * Update the README
 * Fix some spec tests

metadata.rb

@@ -4,4 +4,4 @@
 license          'Apache 2.0'
 description      'LWRPs for managing SSH known_hosts and config files'
 long_description IO.read(File.join(File.dirname(__FILE__), 'README.md'))
-version          '0.10.2'
+version          '0.10.4'

providers/config.rb

@@ -7,10 +7,10 @@ def whyrun_supported?
 end
 
 action :add do
-  unless @new_resource.options.eql? @existing_entries[new_resource.name]
-    @existing_entries[@new_resource.name] = @new_resource.options
+  unless @new_resource.options.eql? @existing_entries[new_resource.host]
+    @existing_entries[@new_resource.host] = @new_resource.options
 
-    converge_by "Adding {@new_resource.name} to #{@path} with #{new_resource.options.inspect}" do
+    converge_by "Adding {@new_resource.host} to #{@path} with #{new_resource.options.inspect}" do
       create_directory
       create_file
     end
@@ -23,9 +23,9 @@ def whyrun_supported?
 
 action :remove do
   if @current_resource.exists?
-    @existing_entries.delete @new_resource.name
+    @existing_entries.delete @new_resource.host
 
-    converge_by "Remove #{@new_resource.name} from #{@path}" do
+    converge_by "Remove #{@new_resource.host} from #{@path}" do
       create_file
     end
   end
@@ -54,6 +54,6 @@ def load_current_resource
   @path = new_resource.path || default_or_user_path(new_resource.user)
   @existing_entries = parse_file @path
 
-  @current_resource = Chef::Resource::SshConfig.new(@new_resource.name)
-  @current_resource.exists = @existing_entries.key? @new_resource.name
+  @current_resource = Chef::Resource::SshConfig.new(@new_resource.host)
+  @current_resource.exists = @existing_entries.key? @new_resource.host
 end

test/cookbooks/ssh_test/recipes/config.rb

@@ -14,7 +14,8 @@
 group 'other_group'
 user 'someone'
 
-ssh_config 'test.io' do
+ssh_config 'something else' do
+  host 'test.io'
   options 'User' => 'testuser', 'DummyKey' => 'I was allowed'
   user  'someone'
   group 'other_group'