Bug 244092

Summary: security/py-fail2ban: Create required directoy on start
Product: Ports & Packages Reporter: theis
Component: Individual Port(s)Assignee: freebsd-ports-bugs (Nobody) <ports-bugs>
Status: Open ---    
Severity: Affects Some People CC: swills, theis
Priority: --- Keywords: buildisok, feature, needs-qa
Version: LatestFlags: theis: maintainer-feedback+
koobs: merge-quarterly?
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch theis: maintainer-approval+

Description theis 2020-02-13 10:45:25 UTC
Created attachment 211607 [details]
patch

Reported at https://github.com/fail2ban/fail2ban/issues/2634
fail2ban should check and, if necessary create, the required directory.

It is still up to the user to ensure that configuration in fail2ban's conf files and FreeBSD's /etc/rc.conf are in sync and that both, pidfile and socket reside in the same directory

Thanks to Florian Koczan
Comment 1 Automation User 2020-02-13 11:05:26 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/117564104
Comment 2 Steve Wills freebsd_committer 2020-02-16 13:59:10 UTC
This looks like a good addition. Have you considered adding -p ${pidfile} to the args, to avoid potential issues with mismatch between rc.conf and the fail2ban config?
Comment 3 theis 2020-02-17 12:41:50 UTC
I thought about that but may main problem is that users have to make sure the directories for pidfile and socket are the same, and here I'm running into conflicts trying to keep 2 configurations in sync. So I just wrote a hint about this.