-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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.
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
add jail.conf and other misc fixes #143
Conversation
The commands are executed through a system shell so shell redirection and process control is allowed. The commands should | ||
return 0 to indicate success which will prevent fail2ban attempting to retry (up to \fBmaxretry\fR times to ban the IP again. | ||
|
||
The following action parameters are in the [defination] section. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WTF... can't post a comment -- comes out blank...
s|defination|Definition|g
ss = os.stat(self.__sock_name) | ||
self.assertIsNot(stat.S_ISSOCK(ss.st_mode), 0) | ||
|
||
# pid removal is done in server.start |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe the asyncServer.start
in server.start
is blocking, such that the PID file should only be removed when AsyncServer (and therefore fail2ban) has stopped.
ok. think I covered everything. Thanks guys. |
I have pushed with some post-merge tune ups. Please checkout master's version on either it is all good ;-) |
jail.conf man page + other misc fixes.
If you want me to separate these kind of pull requests just tell me.