Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

actions.d/bsd-ipfw.conf startstatefile errors in 0.9.0 #713

Closed
ache opened this issue May 2, 2014 · 1 comment
Closed

actions.d/bsd-ipfw.conf startstatefile errors in 0.9.0 #713

ache opened this issue May 2, 2014 · 1 comment
Milestone
0.9.1

Comments

@ache
Copy link
Contributor

ache commented May 2, 2014

bsd-ipfw.conf have

startstatefile = /var/run/fail2ban/ipfw-started-table_<table>

<table> is not expanded here perhaps because of

Values: STRING

 instead of # Values: CMD for startstatefile causing lots of on-stop errors like that: 2014-05-02 17:46:39,615 fail2ban.server.action[74335]: ERROR [ ! -f /var/run/fail2ban/ipfw-started-table_<table> ] || ( read num < "/var/run/fail2ban/ipfw-started-table_<table>" ipfw -q delete $num rm "/var/run/fail2ban/ipfw-started-table_<table>" ) -- stderr: 'cannot open table: No such file or directory\ncann ot open /var/run/fail2ban/ipfw-started-table_<table>: No such file or directory\nipfw: missing rule specification\n rm: /var/run/fail2ban/ipfw-started-table_<table>: No such file or directory\n'
kwirk added a commit to kwirk/fail2ban that referenced this issue May 3, 2014
@kwirk
BF: Tags not fully recursively substituted
b3266ba 
Note: recursive check ignored for "matches", as tags would be escaped,
and hence shouldn't match "<%s>" as "<ip>" would become "\<ip\>". This
therefore maintains advantage of delayed call for {ip,jail,}matches.

Fixes fail2bangh-713
@kwirk kwirk mentioned this issue May 3, 2014
Merged
@kwirk
Copy link
Contributor

kwirk commented May 3, 2014

@ache Thanks for reporting this bug. I've been able to replicate. Should be fixed once #715 is reviewed and hopefully merged 😄

@kwirk kwirk added this to the 0.9.1 milestone May 7, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.9.1
Development

No branches or pull requests
2 participants
@kwirk @ache