gandalf/secubox-openwrt
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(interceptor): Fix RPCD handler shell syntax error

Browse Source 
Remove 'local' keyword from case statement block where it's not
allowed in POSIX shell. Replace && block conditions with proper
if/then/fi statements for health score calculation.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
CyberMind-FR 2026-02-11 11:51:53 +01:00
parent af622285ed
commit 1ab19cb778
1 changed files with 12 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
package/secubox/luci-app-interceptor/root/usr/libexec/rpcd/luci.interceptor
View File

@ -177,38 +177,38 @@ case "$1" in
get_failover_status get_failover_status
# Overall health score (0-100) # Overall health score (0-100)
local score=0 score=0
local pillars_active=0 pillars_active=0
# WPAD # WPAD
[ "$(uci -q get network_tweaks.global.wpad_enforce)" = "1" ] || [ -f /www/wpad.dat ] && { if [ "$(uci -q get network_tweaks.global.wpad_enforce)" = "1" ] || [ -f /www/wpad.dat ]; then
score=$((score + 20)) score=$((score + 20))
pillars_active=$((pillars_active + 1)) pillars_active=$((pillars_active + 1))
} fi
# mitmproxy running # mitmproxy running
pgrep -x mitmproxy >/dev/null 2>&1 || lxc-ls --running 2>/dev/null | grep -q "secbx-mitmproxy" && { if pgrep mitmproxy >/dev/null 2>&1 || lxc-ls --running 2>/dev/null | grep -q "secbx-mitmproxy"; then
score=$((score + 20)) score=$((score + 20))
pillars_active=$((pillars_active + 1)) pillars_active=$((pillars_active + 1))
} fi
# CDN Cache running # CDN Cache running
pgrep squid >/dev/null 2>&1 && { if pgrep squid >/dev/null 2>&1; then
score=$((score + 20)) score=$((score + 20))
pillars_active=$((pillars_active + 1)) pillars_active=$((pillars_active + 1))
} fi
# Cookie Tracker enabled # Cookie Tracker enabled
[ "$(uci -q get cookie-tracker.main.enabled)" = "1" ] && { if [ "$(uci -q get cookie-tracker.main.enabled)" = "1" ]; then
score=$((score + 20)) score=$((score + 20))
pillars_active=$((pillars_active + 1)) pillars_active=$((pillars_active + 1))
} fi
# API Failover enabled # API Failover enabled
[ "$(uci -q get cdn-cache.api_failover.enabled)" = "1" ] && { if [ "$(uci -q get cdn-cache.api_failover.enabled)" = "1" ]; then
score=$((score + 20)) score=$((score + 20))
pillars_active=$((pillars_active + 1)) pillars_active=$((pillars_active + 1))
} fi
json_add_object "summary" json_add_object "summary"
json_add_int "health_score" "$score" json_add_int "health_score" "$score"