Issue #3909 has been updated by Steven Seed.

I haven't been able to get the doc signed, however I would like to remove the 
patch I uploaded as it doesn't work properly. The original posters suggestion 
is better, but you must escape the '.' in the regular expression with a 
backlash... $1.sub(/\.$/, '')
----------------------------------------
Bug #3909: Facter does not behave properly with non-existent top-level domains.
https://projects.puppetlabs.com/issues/3909#change-60572

Author: Joe McDonagh
Status: Accepted
Priority: Normal
Assignee: 
Category: library
Target version: 1.6.x
Keywords: 
Branch: 
Affected Facter version: 


On behalf of a puppet-users posted who did not want to create an additional 
login to file a bug:

I was stumbling over the fact that I use a (not existing) toplevel
domain in my environment. So I set up the dnsdomainname to print out the
correct domain (without fullstop ('.')). Additional I limited the search
path in resolv.conf to end with a '.'.

That seems to tangle facter. As I read the code it needs a '.' anywhere
in domainname to work and the fallback to parse /etc/resolv.conf cannot
handle trailing '.'. The last is easy to handle by $1.sub(/\.$/, '') but
the first I do not know how to handle correctly for every case (At least
on debian there seems to be '(none)' if it is not defined correctly.)

Could that go into upstream code respective how to fix the first case
proper?



-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to