Colin

Not Sure How This Ever Worked

In the header block it mentions a staus check being implemented which I think is where this came from at line 18 ps_state=$(ps aux |grep "fail2ban.sock" |grep -v grep| wc -l) This seems to be an attempt to check if the fail2ban.sock socket has been created by the nagios-server, except this is looking at processes not sockets or at least it is on my Centos 7.7 installation, it always returns zero so the program aborts further down with the message ++++ Process is not running ++++ Change line 18 to this instead ps_state=$(ss -aux |grep "fail2ban.sock" |grep -v grep| wc -l) Basically use ss instead of ps and put the - before the aux options. Then it returns 1 as expected if the fail2ban-server is running and the script works correctly.

Reviewed 6 years ago

Useful For Quick Check On Status

I have been using this for quite a while now on my Android phone, created a link on the phones home screen which opens it up, great for a quick status overview on the move, recently started having that Python warning mentioned in another review did as suggested and its working great again.

Reviewed 7 years ago