Brainsware / puppet-trafficserver

Puppet module to manage trafficserver installation and configuration
https://forge.puppet.com/brainsware/trafficserver
Apache License 2.0
1 stars 7 forks source link

ssl_multicert: fix parsing of lines with missing fields #20

Closed ffrank closed 9 years ago

ffrank commented 9 years ago

Uninitialized is-values on properties seem to result in missing hits during out-of-sync checks [citation needed]. By considering their value to be the empty string, the agent can make correct syncing decisions now.

Note that puppet resource trafficserver_ssl_multicert still works as expected and will not include bogus empty string values in the manifest output.

ffrank commented 9 years ago

Fix for issue #18

igalic commented 9 years ago

we may also need this same approach in the other provider(s) that use(s) parsedfile.

igalic commented 9 years ago

then again, given the rather simple structure of trafficserver_plugin, i may be wrong there.