Issue #16686 has been updated by eric sorenson.

Tracker changed from Bug to Feature
Subject changed from File-Serving Configuration parser incorrectly handles 
allow_ip statements in fileserver.conf to File-Serving Configuration parser 
does not implement allow_ip statements in fileserver.conf
Target version set to 3.x

fileserver.conf wasn't updated to support allow_ip, only auth.conf. these are 
separate parsers, unfortunately. auth.conf is parsed in 
`lib/puppet/network/auth_config_parser.rb`

I agree that this should be normalized, but I'm altering the description to 
indicate the source of the problem.

As of today the posted documentation is still for 2.7 not 3.0 so the change is 
not reflected on the website. I'll check back on this issue once the 3.0 docs 
release and make sure this inconsistency is appropriately called out until it's 
fixed.
----------------------------------------
Feature #16686: File-Serving Configuration parser does not implement allow_ip 
statements in fileserver.conf
https://projects.puppetlabs.com/issues/16686#change-72472

Author: Wolfgang Miedl
Status: Unreviewed
Priority: Normal
Assignee: 
Category: fileserving
Target version: 3.x
Affected Puppet version: 3.0.0
Keywords: 
Branch: 


In the current 3.0.0 release, the file serving configuration parser incorrectly 
handles "allow_ip" statements in fileserver.conf. Both an allow and allow_ip 
statement will result in Puppet::FileServing::Configuration::Parser.allow being 
called, which again calls Puppet::Network::AuthStore.allow.

This will raise an AuthStoreError in case of an allow_ip statement, as 
Puppet::Network::AuthStore::Declaration.parse fails to parse the parameter. The 
fix is to call Puppet::Network::AuthStore.allow_ip instead in case an allow_ip 
statement is read, which will delegate the parsing to the correct method 
(Puppet::Network::AuthStore::Declaration.parse_ip)

The attached diff illustrates the issue and a possible fix.


-- 
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