Merge pull request #156 from andrekeller/feature-server-nobind

feature (server): add nobind parameter support.
voxpupuli · May 12, 2015 · baffc40 · baffc40
2 parents 6ae2350 + 50ee23c
commit baffc40
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 0 deletions.
diff --git a/manifests/server.pp b/manifests/server.pp
@@ -314,6 +314,10 @@
 #   with client configuration
 #   Default: true
 #
+# [*nobind*]
+#   Boolean. Whether or not to bind to a specific port number.
+#   Default: false
+#
 # [*custom_options*]
 #   Hash of additional options that you want to append to the configuration file.
 #
@@ -426,6 +430,7 @@
   $shared_ca                 = undef,
   $autostart                 = undef,
   $ns_cert_type              = true,
+  $nobind                    = false,
   $custom_options            = {},
 ) {
 

diff --git a/spec/defines/openvpn_server_spec.rb b/spec/defines/openvpn_server_spec.rb
@@ -237,6 +237,7 @@
       'ping_timer_rem'      => true,
       'tls_auth'            => true,
       'tls_client'          => true,
+      'nobind'              => true,
     } }
 
     let(:facts) { {
@@ -267,6 +268,8 @@
     it { should_not contain_file('/etc/openvpn/test_client.conf').with_content(/^dh/) }
     it { should contain_file('/etc/openvpn/test_client.conf').with_content(%r{^tls-client$}) }
     it { should contain_file('/etc/openvpn/test_client.conf').with_content(%r{^key-direction 1$}) }
+    it { should contain_file('/etc/openvpn/test_client.conf').with_content(%r{^nobind$}) }
+    it { should_not contain_file('/etc/openvpn/test_client.conf').with_content(%r{^port\s+\d+$}) }
 
     it { should_not contain_openvpn__ca('test_client') }
   end

diff --git a/templates/server.erb b/templates/server.erb
@@ -27,7 +27,11 @@ proto <%= @proto %>-server
 <% else -%>
 proto <%= @proto %>
 <% end -%>
+<% if @nobind -%>
+nobind
+<% else -%>
 port <%= @port %>
+<% end -%>
 <% if @real_tls_server -%>
 tls-server
 <% end -%>