Puppet

Version: puppet-2.7.18-1.fc17.noarch

How to reproduce:

1) Have a machine without hostname set 2) Try to call puppet parser validate some-file.pp 3) /usr/share/ruby/vendor_ruby/puppet/defaults.rb:225:in <module:Puppet>': undefined methoddowncase' for nil:NilClass (NoMethodError)

    from /usr/share/ruby/vendor_ruby/puppet/defaults.rb:2:in `<top (required)>'

This bit has the bug:

hostname = Facter["hostname"].value
domain = Facter["domain"].value
if domain and domain != ""
fqdn = [hostname, domain].join(".")
else
fqdn = hostname
end
Puppet.setdefaults(
:main,
# We have to downcase the fqdn, because the current ssl stuff (as oppsed to in master) doesn't have good facilities for
# manipulating naming.
:certname => {:default => fqdn.downcase, :desc => "The name to use when handling certificates.  Defaults
to the fully qualified domain name.",
:call_on_define => true, # Call our hook with the default value, so we're always downcased
:hook => proc { |value| raise(ArgumentError, "Certificate names must be lower case; see #1168") unless value == value.downcase }},
:certdnsnames => {
:default => '',
:hook    => proc do |value|

I dont see this in master. Can you please confirm me this was fixed upstream? Thanks

https://github.com/puppetlabs/puppet/blob/master/lib/puppet/defaults.rb