lelutin / puppet-fail2ban

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

Persistant ban function and usedns parameter for reverse dns lookups #17

Closed luisilva closed 8 years ago

luisilva commented 8 years ago

Added the ability to set usedns to (yes, warn or no) and drop a template in for iptables-multiport.conf that adds 3 lines that allow you hold a persistent ban list. It appears this is the way most folks have been doing this.

RE: https://arno0x0x.wordpress.com/2015/12/30/fail2ban-permanent-persistent-bans/

It would be great if you merged this in. If you have any idea about better ways please feel free to let me know. Thanks, Lui

lelutin commented 8 years ago

hi there, thanks for your help with this module!

usedns: yes thanks :)

persistent bans are now builtin with 0.9.x releases with the use of sqlite3. see https://github.com/fail2ban/fail2ban/releases/tag/0.9.0 however, debian jessie still has 0.8.13, so I'll merge it in.

there are some minor details that I can fix on top of your commits:

however, the template file for the multiport action seem to differ from default file:

luisilva commented 8 years ago

Thanks for all the great feed back. I'm happy to work on some of those items and resubmit the request.

luisilva commented 8 years ago

ok, I think we've worked out all the items in you last message. Let me know if there's anything you'd like to add/remove. Thx

luisilva commented 8 years ago

Hey there, I'm going to close this pull request out and resubmit one later on. I'm going to add a fail2ban.local so with can tweak the main config setting as well. I figure I can spare you the commit noise. Thanks!