luci-app-adblock: sync with release 3.6.3

* the DNS Report now displays the hostname, MAC-Address or
  client IP (CLI & LuCI)
* Filter the DNS Query result set for a particular domain, client or
  time frame (CLI & LuCI)
* remove needless XHR.Poll-Events from Reporting page in LuCI
* remove needless 'force sort' option in LuCI

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

3 files changed, 138 insertions, 58 deletions

diff --git a/applications/luci-app-adblock/luasrc/controller/adblock.lua b/applications/luci-app-adblock/luasrc/controller/adblock.lua
index e5061bc9d9..4641ac93bb 100644
--- a/applications/luci-app-adblock/luasrc/controller/adblock.lua
+++ b/applications/luci-app-adblock/luasrc/controller/adblock.lua
@@ -1,4 +1,4 @@
--- Copyright 2017-2018 Dirk Brenken (dev@brenken.org)
+-- Copyright 2017-2019 Dirk Brenken (dev@brenken.org)
 -- This is free software, licensed under the Apache License, Version 2.0
 
 module("luci.controller.adblock", package.seeall)
@@ -31,7 +31,20 @@ function index()
 	entry({"admin", "services", "adblock", "action"}, call("adb_action"), nil).leaf = true
 end
 
-function adb_action(name, domain)
+function adb_action(name, ...)
+	local domain = select(1, ...) or ""
+	local search = select(2, ...) or "+"
+	local count  = select(3, ...) or "50"
+	local filter = select(4, ...) or "false"
+	local print  = select(5, ...) or "false"
+
+	local report_params = {
+		search,
+		count,
+		filter,
+		print
+	}
+
 	if name == "do_suspend" then
 		luci.sys.call("/etc/init.d/adblock suspend >/dev/null 2>&1")
 	elseif name == "do_resume" then
@@ -45,11 +58,17 @@ function adb_action(name, domain)
 			until nixio.fs.readfile(pid_file) == ""
 		end
 	elseif name == "do_report" then
-		luci.sys.call("/etc/init.d/adblock report false >/dev/null 2>&1")
+		luci.sys.call("/etc/init.d/adblock report " ..table.concat(report_params, " ").. " >/dev/null 2>&1")
 		local rep_dir  = uci:get("adblock", "extra", "adb_repdir") or "/tmp"
 		repeat
 			nixio.nanosleep(1)
 		until not nixio.fs.access(rep_dir.. "/adb_report.raw")
+	elseif name == "do_filter" then
+		luci.sys.call("/etc/init.d/adblock report " ..table.concat(report_params, " ").. " >/dev/null 2>&1")
+		local rep_dir  = uci:get("adblock", "extra", "adb_repdir") or "/tmp"
+		repeat
+			nixio.nanosleep(1)
+		until nixio.fs.access(rep_dir.. "/adb_report.final")
 	elseif name == "add_blacklist" then
 		local file = uci:get("adblock", "blacklist", "adb_src") or "/etc/adblock/adblock.blacklist"
 		if nixio.fs.access(file) then
@@ -110,7 +129,7 @@ function report_text()
 	local content
 
 	rep_dir  = uci:get("adblock", "extra", "adb_repdir") or "/tmp"
-	rep_file = rep_dir.. "/adb_report"
+	rep_file = rep_dir.. "/adb_report.final"
 	http.prepare_content("text/plain")
 
 	if nixio.fs.access(rep_file) then
diff --git a/applications/luci-app-adblock/luasrc/model/cbi/adblock/overview_tab.lua b/applications/luci-app-adblock/luasrc/model/cbi/adblock/overview_tab.lua
index edf5df7ec9..3a37b45dfb 100644
--- a/applications/luci-app-adblock/luasrc/model/cbi/adblock/overview_tab.lua
+++ b/applications/luci-app-adblock/luasrc/model/cbi/adblock/overview_tab.lua
@@ -1,4 +1,4 @@
--- Copyright 2017-2018 Dirk Brenken (dev@brenken.org)
+-- Copyright 2017-2019 Dirk Brenken (dev@brenken.org)
 -- This is free software, licensed under the Apache License, Version 2.0
 
 local fs   = require("nixio.fs")
@@ -222,9 +222,4 @@ e29.datatype = "range(1,10)"
 e29.default = 1
 e29.optional = true
 
-e30 = e:option(Flag, "adb_forcesrt", translate("Force Overall Sort"),
-	translate("Enable memory intense overall sort / duplicate removal on low memory devices (< 64 MB free RAM)"))
-e30.optional = true
-e30.default = nil
-
 return m
diff --git a/applications/luci-app-adblock/luasrc/view/adblock/report.htm b/applications/luci-app-adblock/luasrc/view/adblock/report.htm
index 4e3d65ba75..814cc2fb25 100644
--- a/applications/luci-app-adblock/luasrc/view/adblock/report.htm
+++ b/applications/luci-app-adblock/luasrc/view/adblock/report.htm
@@ -1,5 +1,5 @@
 <%#
-Copyright 2017-2018 Dirk Brenken (dev@brenken.org)
+Copyright 2017-2019 Dirk Brenken (dev@brenken.org)
 This is free software, licensed under the Apache License, Version 2.0
 -%>
 
@@ -84,10 +84,10 @@ This is free software, licensed under the Apache License, Version 2.0
 		s += '<div class="th left"><%:Domain%></div>';
 		s += '<div class="th left"><%:Answer%></div>';
 		s += '<div class="th left"><%:Action%></div></div>';
-		var btn;
+		var btn1;
 		var record;
 		var title_arr = ["<%:Date%>", "<%:Time%>", "<%:Client%>", "<%:Domain%>", "<%:Answer%>", "<%:Action%>"];
-		var array = text.split("\n", 50);
+		var array = text.split("\n");
 		for (var i=0;i<array.length;i++)
 		{
 			record = array[i].split("\t");
@@ -99,14 +99,18 @@ This is free software, licensed under the Apache License, Version 2.0
 					s += '<div class="td left" data-title="' + title_arr[j] + '">' + record[j] + '</div>';
 					if (record[4] === "NX")
 					{
-						btn = '<div class="td left" data-title="' + title_arr[5] + '"><input type="button" class="cbi-button cbi-button-edit" name="add_whitelist,' + record[3] + '" value="<%:Whitelist%>" onclick="btn_action(this)" /></div>';
+						btn1 = '<div class="td left" data-title="' + title_arr[5] + '"><input type="button" class="cbi-button cbi-button-edit" name="add_whitelist,' + record[3] + '" value="<%:Whitelist%>" onclick="btn_action(this)" /></div>';
+					}
+					else if (record[4] === "OK")
+					{
+						btn1 = '<div class="td left" data-title="' + title_arr[5] + '"><input type="button" class="cbi-button cbi-button-remove" name="add_blacklist,' + record[3] + '" value="<%:Blacklist%>" onclick="btn_action(this)" /></div>';
 					}
 					else
 					{
-						btn = '<div class="td left" data-title="' + title_arr[5] + '"><input type="button" class="cbi-button cbi-button-remove" name="add_blacklist,' + record[3] + '" value="<%:Blacklist%>" onclick="btn_action(this)" /></div>';
+						btn1 = '-'
 					}
 				}
-				s += btn + '</div>'
+				s += btn1 + '</div>'
 			}
 		}
 		document.getElementById("value_6").innerHTML = s;
@@ -118,27 +122,96 @@ This is free software, licensed under the Apache License, Version 2.0
 
 		if (action[0] === "do_report")
 		{
-			var btn         = document.getElementById("btn");
-			var btn_running = document.getElementById("btn_running");
+			var btn1         = document.getElementById("btn1");
+			var btn1_running = document.getElementById("btn1_running");
 
-			btn.disabled = true;
-			running(btn_running, 1);
-			action[1] = "-"
-		}
+			btn1.disabled = true;
+			running(btn1_running, 1);
+
+			document.getElementById("filter_search").value = '';
+			document.getElementById("filter_count").selectedIndex = 1;
 
-		new XHR.get('<%=luci.dispatcher.build_url("admin", "services", "adblock")%>/action/' + action[0] + "/" + action[1], null,
-		function(x)
+			XHR.get('<%=luci.dispatcher.build_url("admin", "services", "adblock")%>/action/' + action[0], null,
+			function(x)
+			{
+				if (!x)
+				{
+					return;
+				}
+				XHR.get('<%=luci.dispatcher.build_url("admin", "services", "adblock", "report_json")%>', null,
+				function(x, json_info)
+				{
+					if (!x || !json_info || !json_info.data)
+					{
+						running(btn1_running, 0);
+						btn1.disabled = false;
+						return;
+					}
+					report_json(json_info);
+				});
+				XHR.get('<%=luci.dispatcher.build_url("admin", "services", "adblock", "report_text")%>', null,
+				function(x)
+				{
+					if (!x || !x.responseText)
+					{
+						return;
+					}
+					report_text(x.responseText);
+					running(btn1_running, 0);
+					btn1.disabled = false;
+				});
+			});
+		}
+		else if (action[0] === "do_filter")
 		{
-			if (!x)
+			var btn2         = document.getElementById("btn2");
+			var btn2_running = document.getElementById("btn2_running");
+			var search       = document.getElementById("filter_search").value.replace(/[^\x00-\x7F]|[\s!@#$%^&*()+=\[\]{};'"\\|,<>\/?]/g,'') || "\"+\"";
+			var count        = document.getElementById("filter_count").value;
+
+			btn2.disabled = true;
+			running(btn2_running, 1);
+			if (search != "\"+\"")
 			{
-				return;
+				document.getElementById("filter_search").value = search;
 			}
-			if (action[0] === "do_report")
+
+			XHR.get('<%=luci.dispatcher.build_url("admin", "services", "adblock")%>/action/' + action[0] + "/" + "-" + "/" + search + "/" + count + "/" + "true" + "/" + "false", null,
+			function(x)
 			{
-				running(btn_running, 0);
-				btn.disabled = false;
-			}
-		});
+				if (!x)
+				{
+					return;
+				}
+				XHR.get('<%=luci.dispatcher.build_url("admin", "services", "adblock", "report_text")%>', null,
+				function(x)
+				{
+					if (!x || !x.responseText)
+					{
+						return;
+					}
+					report_text(x.responseText);
+					running(btn2_running, 0);
+					btn2.disabled = false;
+				});
+			});
+		}
+		else if (action[0] === "add_blacklist" || action[0] === "add_whitelist")
+		{
+			XHR.get('<%=luci.dispatcher.build_url("admin", "services", "adblock")%>/action/' + action[0] + "/" + action[1], null,
+			function(x)
+			{
+				if (!x)
+				{
+					return;
+				}
+				btn3 = document.getElementsByName(value.name);
+				for (var i=0; i<btn3.length; i++)
+				{
+					btn3[i].disabled = true;
+				}
+			});
+		}
 	}
 
 	function running(element, state)
@@ -159,20 +232,8 @@ This is free software, licensed under the Apache License, Version 2.0
 	{
 		if (!x || !json_info || !json_info.data)
 		{
-			running(btn_running, 0);
-			btn.disabled = false;
-			return;
-		}
-		report_json(json_info);
-	});
-
-	XHR.poll(-1, '<%=luci.dispatcher.build_url("admin", "services", "adblock", "report_json")%>', null,
-	function(x, json_info)
-	{
-		if (!x || !json_info || !json_info.data)
-		{
-			running(btn_running, 0);
-			btn.disabled = false;
+			running(btn1_running, 0);
+			btn1.disabled = false;
 			return;
 		}
 		report_json(json_info);
@@ -187,16 +248,6 @@ This is free software, licensed under the Apache License, Version 2.0
 		}
 		report_text(x.responseText);
 	});
-
-	XHR.poll(-1, '<%=luci.dispatcher.build_url("admin", "services", "adblock", "report_text")%>', null,
-	function(x)
-	{
-		if (!x || !x.responseText)
-		{
-			return;
-		}
-		report_text(x.responseText);
-	});
 //]]>
 </script>
 
@@ -227,8 +278,8 @@ This is free software, licensed under the Apache License, Version 2.0
 	</div>
 	<br />
 	<div id="button">
-		<input type="button" class="cbi-button cbi-button-action important" id="btn" name="do_report" value="<%:Refresh Report%>" onclick="btn_action(this)" />
-		<span id="btn_running" class="btn_running"></span>
+		<input type="button" class="cbi-button cbi-button-action important" id="btn1" name="do_report" value="<%:Refresh Report%>" onclick="btn_action(this)" />
+		<span id="btn1_running" class="btn1_running"></span>
 	</div>
 </div>
 <br />
@@ -236,8 +287,23 @@ This is free software, licensed under the Apache License, Version 2.0
 	<h3><%:Top 10 Reporting%></h3>
 	<div class="table" id="value_5"></div>
 </div>
-<br />
+<hr />
 <div class="cbi-section">
+	<div class="cbi-section-descr"><%:Filter the DNS Query result set for a particular domain, client or time frame.%></div>
+	<div style="float:left;">
+		<input type="text" placeholder="<%:Domain/Client/Date/Time%>" id="filter_search" name="filter_search" />
+		<select name="filter_count" id="filter_count">
+			<option value="25">25</option>
+			<option value="50" selected="selected">50</option>
+			<option value="100">100</option>
+			<option value="250">250</option>
+			<option value="500">500</option>
+		</select>
+		<input type="button" class="cbi-button cbi-button-action" id="btn2" name="do_filter" value="<%:Filter%>" onclick="btn_action(this)" />
+		<span id="btn2_running" class="btn2_running"></span>
+	</div>
+	<br />
+	<br />
 	<h3><%:Latest DNS Queries%></h3>
 	<div class="table" id="value_6"></div>
 </div>