adblock: update 3.8.8

* print to stdout if 'logger' is not available
* small fixes

Signed-off-by: Dirk Brenken <dev@brenken.org>

diff --git a/net/adblock/Makefile b/net/adblock/Makefile
index 7d849f96486055cbddba4ca51d70b190745c6ac9..24c5777d1005ae00e29407927603ee153e207b8b 100644 (file)
--- a/net/adblock/Makefile
+++ b/net/adblock/Makefile
@@ -6,7 +6,7 @@
 include $(TOPDIR)/rules.mk
 
 PKG_NAME:=adblock
-PKG_VERSION:=3.8.7
+PKG_VERSION:=3.8.8
 PKG_RELEASE:=1
 PKG_LICENSE:=GPL-3.0-or-later
 PKG_MAINTAINER:=Dirk Brenken <dev@brenken.org>

diff --git a/net/adblock/files/adblock.mail b/net/adblock/files/adblock.mail
index 0f8b1766100f3f9fb373c858e8c9dfef9781a378..a0b8c3f0b014416b0dcd3c7ffd84d575ec231384 100755 (executable)
--- a/net/adblock/files/adblock.mail
+++ b/net/adblock/files/adblock.mail
@@ -21,6 +21,8 @@ then
 fi
 adb_ver="${1}"
 adb_mail="$(command -v msmtp)"
+adb_logger="$(command -v logger)"
+adb_logread="$(command -v logread)"
 adb_rc=1
 
 if [ "${adb_debug}" -eq 1 ]
@@ -28,13 +30,6 @@ then
        debug="--debug"
 fi
 
-# mail header & receiver check
-#
-if [ -z "${adb_mreceiver}" ]
-then
-       logger -p "err" -t "adblock-${adb_ver}[${$}]" "please set the mail receiver with the 'adb_mreceiver' option"
-       exit ${adb_rc}
-fi
 adb_mhead="From: ${adb_msender}\\nTo: ${adb_mreceiver}\\nSubject: ${adb_mtopic}\\nReply-to: ${adb_msender}\\nMime-Version: 1.0\\nContent-Type: text/html\\nContent-Disposition: inline\\n\\n"
 
 # info preparation
@@ -44,8 +39,9 @@ adb_info="$(/etc/init.d/adblock status 2>/dev/null)"
 if [ -f "/var/log/messages" ]
 then
        log_info="$(awk '/adblock-/{NR=1;max=79;if(length($0)>max+1)while($0){if(NR==1){print substr($0,1,max),"↵"} else {print " ",substr($0,1,max)}{$0=substr($0,max+1);NR=NR+1}}else print}' /var/log/messages)"
-else
-       log_info="$(logread -e "adblock-" | awk '{NR=1;max=79;if(length($0)>max+1)while($0){if(NR==1){print substr($0,1,max),"↵"} else {print " ",substr($0,1,max)}{$0=substr($0,max+1);NR=NR+1}}else print}')"
+elif [ -x "${adb_logread}" ]
+then
+       log_info="$("${adb_logread}" -e "adblock-" | awk '{NR=1;max=79;if(length($0)>max+1)while($0){if(NR==1){print substr($0,1,max),"↵"} else {print " ",substr($0,1,max)}{$0=substr($0,max+1);NR=NR+1}}else print}')"
 fi
 
 # mail body
@@ -58,13 +54,24 @@ adb_mtext="${adb_mtext}</pre></body></html>"
 
 # send mail
 #
-if [ -x "${adb_mail}" ]
+if [ -x "${adb_mail}" ] && [ -n "${adb_mreceiver}" ]
 then
        printf "%b" "${adb_mhead}${adb_mtext}" 2>/dev/null | "${adb_mail}" ${debug} -a "${adb_mprofile}" "${adb_mreceiver}" >/dev/null 2>&1
        adb_rc=${?}
-       logger -p "info" -t "adblock-${adb_ver}[${$}]" "mail sent to '${adb_mreceiver}' with rc '${adb_rc}'"
+       class="info"
+       log_msg="mail sent to '${adb_mreceiver}' with rc '${adb_rc}'"
+elif [ -z "${adb_mreceiver}" ]
+then
+       class="err"
+       log_msg="please set the mail receiver with the 'adb_mreceiver' option"
 else
-       logger -p "err" -t "adblock-${adb_ver}[${$}]" "msmtp mail daemon not found"
+       class="err"
+       log_msg="msmtp mail daemon not found"
+fi
+if [ -x "${adb_logger}" ]
+then
+       "${adb_logger}" -p "${class}" -t "adblock-${adb_ver}[${$}]" "${log_msg}"
+else
+       printf "%s %s %s\\n" "${class}" "adblock-${adb_ver}[${$}]" "${log_msg}"
 fi
-
 exit ${adb_rc}

diff --git a/net/adblock/files/adblock.service b/net/adblock/files/adblock.service
index c6ac40a0461f0d9d0b41e76ff7517b3b01fac2d8..0a9c9ce27ead55e4b58a0e88820062bffea1f738 100755 (executable)
--- a/net/adblock/files/adblock.service
+++ b/net/adblock/files/adblock.service
@@ -16,12 +16,22 @@ then
 fi
 adb_ver="${1}"
 adb_ubus="$(command -v ubus)"
+adb_logger="$(command -v logger)"
 
 if [ -x "${adb_ubus}" ] && [ -n "${adb_dns}" ]
 then
-       logger -p "info" -t "adblock-${adb_ver}[${$}]" "ubus/adblock service started"
        "${adb_ubus}" -S -M r -m invoke monitor | \
                { grep -qE "\"method\":\"(set|signal)\",\"data\":\{\"name\":\"${adb_dns}\""; [ $? -eq 0 ] && /etc/init.d/adblock start; }
+       class="info"
+       log_msg="ubus/adblock service started"
 else
-       logger -p "err" -t "adblock-${adb_ver}[${$}]" "can't start ubus/adblock service"
+       class="err"
+       log_msg="can't start ubus/adblock service"
 fi
+if [ -x "${adb_logger}" ]
+then
+       "${adb_logger}" -p "${class}" -t "adblock-${adb_ver}[${$}]" "${log_msg}"
+else
+       printf "%s %s %s\\n" "${class}" "adblock-${adb_ver}[${$}]" "${log_msg}"
+fi
+

diff --git a/net/adblock/files/adblock.sh b/net/adblock/files/adblock.sh
index 0702da9602554e1c91f87dd034a667c92f9c73a9..2b724f260e53d5fced453e2c486d232879eb778e 100755 (executable)
--- a/net/adblock/files/adblock.sh
+++ b/net/adblock/files/adblock.sh
@@ -13,7 +13,7 @@
 #
 LC_ALL=C
 PATH="/usr/sbin:/usr/bin:/sbin:/bin"
-adb_ver="3.8.7"
+adb_ver="3.8.8"
 adb_basever=""
 adb_enabled=0
 adb_debug=0
@@ -43,6 +43,7 @@ adb_repdir="/tmp"
 adb_reputil="$(command -v tcpdump)"
 adb_repchunkcnt="5"
 adb_repchunksize="1"
+adb_logger="$(command -v logger)"
 adb_action="${1:-"start"}"
 adb_pidfile="/var/run/adblock.pid"
 adb_ubusservice="/etc/adblock/adblock.service"
@@ -105,6 +106,7 @@ f_load()
        if [ -z "${adb_basever}" ] || [ "${adb_ver%.*}" != "${adb_basever}" ]
        then
                f_log "info" "your adblock config seems to be too old, please update your config with the '--force-maintainer' opkg option"
+               f_rmtemp
                exit 0
        fi
 
@@ -916,13 +918,24 @@ f_log()
 
        if [ -n "${log_msg}" ] && { [ "${class}" != "debug" ] || [ "${adb_debug}" -eq 1 ]; }
        then
-               logger -p "${class}" -t "adblock-${adb_ver}[${$}]" "${log_msg}"
+               if [ -x "${adb_logger}" ]
+               then
+                       "${adb_logger}" -p "${class}" -t "adblock-${adb_ver}[${$}]" "${log_msg}"
+               else
+                       printf "%s %s %s\\n" "${class}" "adblock-${adb_ver}[${$}]" "${log_msg}"
+               fi
                if [ "${class}" = "err" ]
                then
                        f_rmdns
                        f_bgserv "stop"
                        f_jsnup "error"
-                       logger -p "${class}" -t "adblock-${adb_ver}[${$}]" "Please also check 'https://github.com/openwrt/packages/blob/master/net/adblock/files/README.md'"
+                       log_msg="Please also check 'https://github.com/openwrt/packages/blob/master/net/adblock/files/README.md'"
+                       if [ -x "${adb_logger}" ]
+                       then
+                               "${adb_logger}" -p "${class}" -t "adblock-${adb_ver}[${$}]" "${log_msg}"
+                       else
+                               printf "%s %s %s\\n" "${class}" "adblock-${adb_ver}[${$}]" "${log_msg}"
+                       fi
                        exit 1
                fi
        fi
@@ -934,11 +947,11 @@ f_bgserv()
 {
        local bg_pid status="${1}"
 
-       bg_pid="$(pgrep -f "^/bin/sh ${adb_ubusservice}|^/bin/ubus -S -M r -m invoke monitor|^grep -qF \"method\":\"set\",\"data\":\\{\"name\":\"${adb_dns}\"" | awk '{ORS=" "; print $1}')"
+       bg_pid="$(pgrep -f "^/bin/sh ${adb_ubusservice}.*|^/bin/ubus -S -M r -m invoke monitor|^grep -qF \"method\":\"set\",\"data\":\\{\"name\":\"${adb_dns}\"" | awk '{ORS=" "; print $1}')"
        if [ -z "${bg_pid}" ] && [ "${status}" = "start" ] \
                && [ -x "${adb_ubusservice}" ] && [ "${adb_dnsfilereset}" = "true" ]
        then
-               ( "${adb_ubusservice}" "${adb_ver}" &)
+               ( "${adb_ubusservice}" "${adb_ver}" & )
        elif [ -n "${bg_pid}" ] && [ "${status}" = "stop" ] 
        then
                kill -HUP "${bg_pid}" 2>/dev/null