Thank you for contributing to this project!
This request allows changing the default port for any service watched by fail2ban. Check Readme.md for the usage of the "ports" parameter.
Most changes made in this request are just ternary operators on templates and the addition of the "ports" parameter hash to init.pp
Replace this comment with a description of your pull request.
Adds feature: Allow modification of default ports for each service
Replace this comment with the list of issues or n/a.
The PR addresses the same issue as #48 but with a bit more generic approach.
Using 2 level Hash might be possible to override basically any attribute in the template.
lookup() in templates might not be the best approach, but I can't think of better alternative. Passing explicitly each variable explicitly to the template would generate loads of code. Another option is to write a custom function for checking key existence in the configuration hash (but it doesn't add much to code readability).
Let me know if you're ok with this, so that I'm able to proceed with modifying rest of jails.
These lines were probably commented out by mistake in refactoring (d9d2d03d396fb1d24ff7602d253c9748c99b9cdc).
Btw. there are no acceptance tests for RedHat, it might be safer to use Centos templates (if there are no substantial differences).