Fixing :bindaddress setting to work with the new server subsystem.

It now automatically switches values depending on server type,
while still allowing overriding.

Signed-off-by: Luke Kanies <luke@madstop.com>

1 files changed, 6 insertions, 3 deletions

diff --git a/lib/puppet/defaults.rb b/lib/puppet/defaults.rb
index ba47dcd25..d0c2eff84 100644
--- a/lib/puppet/defaults.rb
+++ b/lib/puppet/defaults.rb
@@ -313,12 +313,15 @@ module Puppet
         :config => ["$confdir/puppet.conf",
             "The configuration file for #{Puppet[:name]}."],
         :pidfile => ["$rundir/$name.pid", "The pid file"],
-        :bindaddress => ["", "The address to bind to.  Mongrel servers
+        :bindaddress => ["", "The address a listening server should bind to.  Mongrel servers
             default to 127.0.0.1 and WEBrick defaults to 0.0.0.0."],
-        :servertype => ["webrick", "The type of server to use.  Currently supported
+        :servertype => {:default => "webrick", :desc => "The type of server to use.  Currently supported
             options are webrick and mongrel.  If you use mongrel, you will need
             a proxy in front of the process or processes, since Mongrel cannot
-            speak SSL."]
+            speak SSL.",
+            :call_on_define => true, # Call our hook with the default value, so we always get the correct bind address set.
+            :hook => proc { |value|  value == "webrick" ? parent[:bindaddress] = "0.0.0.0" : parent[:bindaddress] = "127.0.0.1" if parent[:bindaddress] == "" }
+        }
     )
 
     self.setdefaults(:puppetmasterd,