Try to find why we have unknown status in some cases

Ysurac · Sep 20, 2024 · 8ae235c · 8ae235c
1 parent 44b8789
commit 8ae235c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/luci-app-openmptcprouter/luasrc/view/openmptcprouter/wanstatus.htm b/luci-app-openmptcprouter/luasrc/view/openmptcprouter/wanstatus.htm
@@ -445,13 +445,13 @@
 							cssc = 'rgb(144, 240, 144)';
 							statusIcon = "<%=resource%>/openmptcprouter/images/statusWarning.png";
 							statusMessageClass = "warning";
-							statusMessage = "Some connectivity tests failed";
+							statusMessage = "Some connectivity tests failed" + '<br />';
 						default:
 							stat = 'Unknown';
 							cssc = 'rgb(144, 240, 144)';
 							statusIcon = "<%=resource%>/openmptcprouter/images/statusWarning.png";
 							statusMessageClass = "warning";
-							statusMessage = "Unknown status";
+							statusMessage = "Unknown status (" + mArray.wans[i].status + ")" + '<br />';
 					}
 					// Populate info
 					var wanip    = mArray.wans[i].wanip;

diff --git a/openmptcprouter-api/files/usr/libexec/rpcd/openmptcprouter b/openmptcprouter-api/files/usr/libexec/rpcd/openmptcprouter
@@ -1273,6 +1273,7 @@ function interfaces_status()
 	    local mac = section ["macaddr"] or ""
 	    local itype = section ["type"] or ""
 	    local state = uci:get("openmptcprouter", interface, "state") or ""
+	    local connectivity = "OK"
 
 	    --if not ipaddr or not gateway then return end
 	    -- Don't show if0 in the overview
@@ -1311,7 +1312,7 @@ function interfaces_status()
 
 	    if enabled == "0" then return end
 
-	    local connectivity = "OK"
+	    --local connectivity = "OK"
 	    if ipaddr == "" and ifname ~= nil and ifname ~= "" and proto ~= "dhcpv6" then
 		    ipaddr = ut.trim(sys.exec("ip -4 -br addr ls dev " .. ifname .. " | awk -F'[ /]+' '{print $3}' | tr -d '\n'"))
 	    end