Merge pull request #330 from wtfo-guru/fix_client_specific_option_push

use double quotes on all push options

Author: Dan33l

spec/defines/openvpn_client_specific_config_spec.rb

@@ -51,8 +51,8 @@
         it { is_expected.to contain_file('/etc/openvpn/test_server/client-configs/test_client').with_content(%r{^iroute 10.0.1.0 255.255.255.0$}) }
         it { is_expected.to contain_file('/etc/openvpn/test_server/client-configs/test_client').with_content(%r{^iroute-ipv6 2001:db8:1234::/64$}) }
         it { is_expected.to contain_file('/etc/openvpn/test_server/client-configs/test_client').with_content(%r{^ifconfig-push 10.10.10.2 255.255.255.0$}) }
-        it { is_expected.to contain_file('/etc/openvpn/test_server/client-configs/test_client').with_content(%r{^push dhcp-option DNS 8.8.8.8$}) }
-        it { is_expected.to contain_file('/etc/openvpn/test_server/client-configs/test_client').with_content(%r{^push redirect-gateway def1$}) }
+        it { is_expected.to contain_file('/etc/openvpn/test_server/client-configs/test_client').with_content(%r{^push "dhcp-option DNS 8.8.8.8"$}) }
+        it { is_expected.to contain_file('/etc/openvpn/test_server/client-configs/test_client').with_content(%r{^push "redirect-gateway def1"$}) }
         it { is_expected.to contain_file('/etc/openvpn/test_server/client-configs/test_client').with_content(%r{^push "route 10.200.100.0 255.255.255.0 10.10.10.1"$}) }
       end
     end

templates/client_specific_config.erb

@@ -8,10 +8,10 @@ iroute-ipv6 <%= iroute_ipv6 %>
 ifconfig-push <%= ifconfig %>
 <% end -%>
 <% @dhcp_options.each do |option| -%>
-push dhcp-option <%= option %>
+push "dhcp-option <%= option %>"
 <% end -%>
 <% if @redirect_gateway -%>
-push redirect-gateway def1
+push "redirect-gateway def1"
 <% end -%>
 <% @route.each do |route| -%>
 push "route <%= route %>"