Remove default empty string parameters

[traylenator]

Pull Request (PR) description

For the defined types and parameters

• For squid::cache_dir and the parameter options the default is now undef rather than ''
• For squid::snmp_port and the parameter options the default is now undef rather than ''
• For squid::refresh_pattern and the parameter options the default is now undef rather than ''
• For squid::http_port and the parameter options the default is now undef rather than ''
• For squid::http_ports and the parameter options the default is now undef rather than ''

This is backwards compatible for all cases except when options has been explicitly set to '' but I feel that is a non-case?

e.g.

squid::http_port{80:
  options => '',

would fail.

Resolves the linter issues for params_empty_string_assignment.

Extra commits include:

• modulesync 5.1.0 templates
• rubocop corrections
• Addition of docs SSLBump type

[traylenator]

Think I read accept' tests for 8 are currently broken somewhere....

[traylenator]

After After https://github.com/voxpupuli/puppet-squid/pull/170