Fix voxpupuliGH-205 raise exception on bad source paramters

nanliu · Aug 31, 2016 · 0246daa · 0246daa
1 parent b9d3bad
commit 0246daa
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/examples/bad_source.pp b/examples/bad_source.pp
@@ -0,0 +1,5 @@
+archive { '/tmp/jta-1.1.jar':
+  ensure  => present,
+  source  => $bad_variable,
+}
+
diff --git a/lib/puppet/provider/archive/ruby.rb b/lib/puppet/provider/archive/ruby.rb
@@ -131,6 +131,8 @@ def transfer_download(archive_filepath)
       FileUtils.copy(Puppet::Util.uri_to_path(uri), temppath)
     when %r{^s3}
       s3_download(temppath)
+    when nil
+      raise(Puppet::Error, "Unable to fetch archive, the source parameter is nil.")
     else
       raise(Puppet::Error, "Source file: #{resource[:source]} does not exists.") unless File.exist?(resource[:source])
       FileUtils.copy(resource[:source], temppath)