Use shellquote to properly escape the username/password for jenkins-s…

…lave defaults

References voxpupuli#374
Fixes voxpupuli#371

manifests/slave.pp

@@ -77,7 +77,7 @@
   $slave_name               = undef,
   $description              = undef,
   $masterurl                = undef,
-  $autodiscoveryaddress      = undef,
+  $autodiscoveryaddress     = undef,
   $ui_user                  = undef,
   $ui_pass                  = undef,
   $version                  = $jenkins::params::swarm_version,
@@ -99,6 +99,9 @@
 
   $client_jar = "swarm-client-${version}-jar-with-dependencies.jar"
   $client_url = "http://maven.jenkins-ci.org/content/repositories/releases/org/jenkins-ci/plugins/swarm-client/${version}/"
+  $quoted_ui_user = shellquote($ui_user)
+  $quoted_ui_pass = shellquote($ui_pass)
+
 
   if $install_java and ($::osfamily != 'Darwin') {
     # Currently the puppetlabs/java module doesn't support installing Java on

spec/classes/jenkins_slave_spec.rb

@@ -44,6 +44,25 @@
         with_content(/--toolLocation Python-2.7=\/usr\/bin\/python2.7/).
         with_content(/--toolLocation Java-1.8=\/usr\/bin\/java/) }
     end
+
+    describe 'with a UI user/password' do
+      let(:user) { '"frank"' }
+      let(:password) { "abignale's" }
+      let(:params) do
+        {
+          :ui_user => user,
+          :ui_pass => password,
+        }
+      end
+
+      it 'should escape the user' do
+        should contain_file(slave_runtime_file).with_content(/^JENKINS_USERNAME='#{user}'$/)
+      end
+
+      it 'should escape the password' do
+        should contain_file(slave_runtime_file).with_content(/^JENKINS_PASSWORD="#{password}"$/)
+      end
+    end
   end
 
   shared_examples 'using slave_name' do

templates/jenkins-slave-defaults.erb

@@ -52,9 +52,9 @@ FSROOT="<%= @slave_home -%>"
 
 DESCRIPTION="<%= @description -%>"
 
-# credentials should be single quoted
-JENKINS_USERNAME="'<%= @ui_user.gsub!(/^\'|\'?$/, '') -%>'"
-JENKINS_PASSWORD="'<%= @ui_pass.gsub!(/^\'|\'?$/, '') -%>'"
+# credentials should be sinjle quoted
+JENKINS_USERNAME=<%= @quoted_ui_user %>
+JENKINS_PASSWORD=<%= @quoted_ui_pass %>
 
 OTHER_ARGS="<%= '-disableSslVerification' if @disable_ssl_verification -%>"