Browse Source

banip: update nginx matching

Currently banip matches nginx log entries starting with
nginx[number]:...

I am running a containerized nginx with alpine as base, which
ends up adding log entries without [number] part..
like this:
nginx:...

This patch updates regex for nginx log entry search to include
both versions.

Signed-off-by: Oskari Rauta <oskari.rauta@gmail.com>
lilik-openwrt-22.03
Oskari Rauta 3 years ago
parent
commit
1441366364
2 changed files with 2 additions and 2 deletions
  1. +1
    -1
      net/banip/Makefile
  2. +1
    -1
      net/banip/files/banip.sh

+ 1
- 1
net/banip/Makefile View File

@ -7,7 +7,7 @@ include $(TOPDIR)/rules.mk
PKG_NAME:=banip PKG_NAME:=banip
PKG_VERSION:=0.7.10 PKG_VERSION:=0.7.10
PKG_RELEASE:=3
PKG_RELEASE:=4
PKG_LICENSE:=GPL-3.0-or-later PKG_LICENSE:=GPL-3.0-or-later
PKG_MAINTAINER:=Dirk Brenken <dev@brenken.org> PKG_MAINTAINER:=Dirk Brenken <dev@brenken.org>


+ 1
- 1
net/banip/files/banip.sh View File

@ -813,7 +813,7 @@ f_bgsrv() {
ban_search="${ban_search}luci: failed login|" ban_search="${ban_search}luci: failed login|"
fi fi
if printf "%s\n" "${ban_logterms}" | grep -q "nginx"; then if printf "%s\n" "${ban_logterms}" | grep -q "nginx"; then
ban_search="${ban_search}nginx\[[0-9]+\]:.*\[error\].*open().*client: [[:alnum:].:]+|"
ban_search="${ban_search}nginx(\[[0-9]+\])?:.*\[error\].*open().*client: [[:alnum:].:]+|"
fi fi
( (
"${ban_logservice}" "${ban_search%?}" & "${ban_logservice}" "${ban_search%?}" &


Loading…
Cancel
Save