search

lelutin / puppet-fail2ban

Manage fail2ban and its jails with puppet
GNU General Public License v3.0
8 stars 31 forks source link

Add support for Centos/Redhat 8 #43

Closed xhejtman closed 4 years ago

xhejtman commented 4 years ago

class fail2ban::config {

if $facts['os']['release']['major'] == '8' {

I suppose, this is meant for debian 8 only, but it collides with Centos 8, so should be fixed like: if $facts['os']['release']['major'] == '8' and $facts['os']['family'] == 'Debian' {

lelutin commented 4 years ago

ah yes you're right!

I've pushed cd0635a76f656705d199a12940f44b9034274749 and 8bb58c97b9595ddef709b1590a0073bf393dceee (since I forgot one if block in the first commit) which should make sure that centos/redhat 8 don't use debian jessie hacks.

I'm tentatively adding support for version 8 of those distros in metadata.json. If you encounter any other bug, let me know :)