Fix MODULES-3158 (any string interpreted as SSLCompression on)

[tpdownes]

1. Fix MODULES-3158 (any string interpreted as SSLCompression on)
2. Convert ssl_honorcipherorder to boolean, backport accepting 'on' or 'off'

[tpdownes]

It is worth mentioning that #1397 failed only because the build tests do not accept "SSLCompression on" but do accept "SSLCompression On". The documentation uses lower case and Apache accepts both cases for "(on|off)". I would have the module output more closely match the documentation and modify the build test here and elsewhere to more closely mimic either the documentation or Apache behavior.

But, in the interest of a single commit that passes the build test as written, I have provided this commit in response to Puppetlabs Ticket 3158.

[bmjen]

This looks good. But can you update the README to indicate that the parameter now allows valid values of On, Off, true, false?

[tpdownes]

Documentation is for other people! I've updated the PR with what I think are necessary and useful updates to the relevant sections.

[bmjen]

Thanks @tpdownes !

[igalic]

there's a function for this ;)
https://github.com/puppetlabs/puppetlabs-apache/blob/master/lib/puppet/parser/functions/bool2httpd.rb