MODULES-1744: use bool2httpd for server_signature and trace_enable

[jlambert121]

No description provided.

[hunner]

Perhaps instead we should validate that they are actually on/off, as in https://github.com/puppetlabs/puppetlabs-apache/blob/1.2.0/manifests/init.pp#L101 ?

[hunner]

As the readme would indicate that those are the values that SHOULD be passed, though not explicitly https://github.com/puppetlabs/puppetlabs-apache#trace_enable https://github.com/puppetlabs/puppetlabs-apache#server_signature

[jlambert121]

I thought about validating, but the function already existed so I went with more options for the user. I'm happy to switch it out for validation, bools make more sense to me though.

On Feb 4, 2015, at 14:52, Hunter Haugen notifications@github.com wrote:

Perhaps instead we should validate that they are actually on/off, as in https://github.com/puppetlabs/puppetlabs-apache/blob/1.2.0/manifests/init.pp#L101 ?

—
Reply to this email directly or view it on GitHub.