search

lelutin / puppet-fail2ban

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

draft: reduce diff with debian bullseye #52

Closed anarcat closed 2 years ago

anarcat commented 2 years ago

This does not completely remove it, obviously some things remain, like:

There may also be other changes I forgot, I lost the actual before/after diff, sorry.

This is also untested.

lelutin commented 2 years ago

thanks for this! the diff looks good in a quick visual inspection.

I guess I'll have to run this through the supported OS/releases to see if it breaks anywhere. if it does it's probably an easy fix.

I'll try to run tests in the next couple of days

lelutin commented 2 years ago

I've added some things on top of your branch, mainly documentation for the new parameter and added the $mode param to the jail defined type too since it's even more relevant to have it there.

The tests I ran on debian stretch, buster and bullseye all worked great!

I've spotted some more file differences with fail2ban.conf and jail.conf, I'll be addressing them today and push the results on top of the master branch. Then once that's done I'll add version 11 of debian as a supported version

thanks for your PR!