From f72558b55e66e410e30916ebc0185176c5c46b56 Mon Sep 17 00:00:00 2001 From: Stan Grishin Date: Tue, 24 Sep 2019 09:17:06 -0700 Subject: [PATCH] luci-app-simple-adblock: dnsmasq.ipset option support Signed-off-by: Stan Grishin --- applications/luci-app-simple-adblock/Makefile | 2 +- .../luasrc/model/cbi/simple-adblock.lua | 47 ++++++++++++++----- 2 files changed, 35 insertions(+), 14 deletions(-) diff --git a/applications/luci-app-simple-adblock/Makefile b/applications/luci-app-simple-adblock/Makefile index 57a9367e1..b7d1a79fb 100644 --- a/applications/luci-app-simple-adblock/Makefile +++ b/applications/luci-app-simple-adblock/Makefile @@ -10,7 +10,7 @@ LUCI_TITLE:=Simple Adblock Web UI LUCI_DESCRIPTION:=Provides Web UI for simple-adblock service. LUCI_DEPENDS:=+luci-mod-admin-full +simple-adblock LUCI_PKGARCH:=all -PKG_RELEASE:=35 +PKG_RELEASE:=37 include ../../luci.mk diff --git a/applications/luci-app-simple-adblock/luasrc/model/cbi/simple-adblock.lua b/applications/luci-app-simple-adblock/luasrc/model/cbi/simple-adblock.lua index 3ec4c4399..f4153355d 100644 --- a/applications/luci-app-simple-adblock/luasrc/model/cbi/simple-adblock.lua +++ b/applications/luci-app-simple-adblock/luasrc/model/cbi/simple-adblock.lua @@ -15,13 +15,18 @@ local dispatcher = require "luci.dispatcher" local enabledFlag = uci:get(packageName, "config", "enabled") local command, outputFile, outputCache, outputGzip local targetDNS = uci:get(packageName, "config", "dns") +local checkDnsmasq = sys.call("which dnsmasq >/dev/null 2>&1") == 0 and true +local checkUnbound = sys.call("which unbound >/dev/null 2>&1") == 0 and true +local checkDnsmasqIpset = sys.call("dnsmasq -v 2>/dev/null | grep -q 'no-ipset' || ! dnsmasq -v 2>/dev/null | grep -q -w 'ipset'") ~= 0 + and sys.call("ipset help hash:net >/dev/null 2>&1") and true if not targetDNS or targetDNS == "" then targetDNS = "dnsmasq.servers" end if targetDNS ~= "dnsmasq.addnhosts" and targetDNS ~= "dnsmasq.conf" and - targetDNS ~= "dnsmasq.servers" and targetDNS ~= "unbound.adb_list" then + targetDNS ~= "dnsmasq.ipset" and targetDNS ~= "dnsmasq.servers" and + targetDNS ~= "unbound.adb_list" then targetDNS = "dnsmasq.servers" end @@ -177,13 +182,34 @@ end s:tab("advanced", translate("Advanced Configuration")) -dns = s:taboption("advanced", ListValue, "dns", translate("DNS Service"), translate("Pick the DNS resolution option to create the adblock list for, see the") .. " " - .. [[]] - .. translate("README") .. [[]] .. " " .. translate("for details.")) -dns:value("dnsmasq.addnhosts", translate("DNSMASQ Additional Hosts")) -dns:value("dnsmasq.conf", translate("DNSMASQ Config")) -dns:value("dnsmasq.servers", translate("DNSMASQ Servers File")) -dns:value("unbound.adb_list", translate("Unbound AdBlock List")) +local dns_descr = translate("Pick the DNS resolution option to create the adblock list for, see the") .. " " + .. [[]] + .. translate("README") .. [[]] .. " " .. translate("for details.") + +if not checkDnsmasq then + dns_descr = dns_descr .. "
" .. translate("Please note that") .. " dnsmasq.addnhosts " .. translate("is not supported on this system.") + dns_descr = dns_descr .. "
" .. translate("Please note that") .. " dnsmasq.conf " .. translate("is not supported on this system.") + dns_descr = dns_descr .. "
" .. translate("Please note that") .. " dnsmasq.ipset " .. translate("is not supported on this system.") + dns_descr = dns_descr .. "
" .. translate("Please note that") .. " dnsmasq.servers " .. translate("is not supported on this system.") +elseif not checkDnsmasqIpset then + dns_descr = dns_descr .. "
" .. translate("Please note that") .. " dnsmasq.ipset " .. translate("is not supported on this system.") +end +if not checkUnbound then + dns_descr = dns_descr .. "
" .. translate("Please note that") .. " unbound.adb_list " .. translate("is not supported on this system.") +end + +dns = s:taboption("advanced", ListValue, "dns", translate("DNS Service"), dns_descr) +if checkDnsmasq then + dns:value("dnsmasq.addnhosts", translate("DNSMASQ Additional Hosts")) + dns:value("dnsmasq.conf", translate("DNSMASQ Config")) + if checkDnsmasqIpset then + dns:value("dnsmasq.ipset", translate("DNSMASQ IP Set")) + end + dns:value("dnsmasq.servers", translate("DNSMASQ Servers File")) +end +if checkUnbound then + dns:value("unbound.adb_list", translate("Unbound AdBlock List")) +end dns.default = "dnsmasq.servers" ipv6 = s:taboption("advanced", ListValue, "ipv6_enabled", translate("IPv6 Support"), translate("Add IPv6 entries to block-list.")) @@ -210,11 +236,6 @@ o8:value("0", translate("Do not use simultaneous processing")) o8:value("1", translate("Use simultaneous processing")) o8.default = 1 -o9 = s:taboption("advanced", ListValue, "allow_non_ascii", translate("Allow Non-ASCII characters in DNSMASQ file"), translate("Only enable if your version of DNSMASQ supports the use of Non-ASCII characters, otherwise DNSMASQ will fail to start.")) -o9:value("0", translate("Do not allow Non-ASCII")) -o9:value("1", translate("Allow Non-ASCII")) -o9.default = "0" - o10 = s:taboption("advanced", ListValue, "compressed_cache", translate("Store compressed cache file on router"), translate("Attempt to create a compressed cache of block-list in the persistent memory.")) o10:value("0", translate("Do not store compressed cache")) o10:value("1", translate("Store compressed cache")) -- 2.25.1