fix(newsbin): Fix RPCD handler for BusyBox sh compatibility

- Use functions instead of inline local vars (not allowed in case)
- Use 1/0 instead of true/false for json_add_boolean
- Use full paths for lxc-info and curl

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
CyberMind-FR 2026-03-14 17:36:46 +01:00
parent 00da717ea4
commit 169b39ed57

View File

@ -13,65 +13,59 @@ get_sab_api() {
grep "^api_key" "$SAB_CONFIG" 2>/dev/null | cut -d'=' -f2 | tr -d ' ' grep "^api_key" "$SAB_CONFIG" 2>/dev/null | cut -d'=' -f2 | tr -d ' '
} }
case "$1" in do_status() {
list) json_init
echo '{"status":{},"queue":{},"history":{},"search":{"query":"string"},"add_nzb":{"url":"string","category":"string"},"pause":{},"resume":{},"remove":{"nzo_id":"string"},"config":{}}'
;;
call)
case "$2" in
status)
json_init
# SABnzbd status # SABnzbd status
local sab_running="false" sab_running=0
local sab_speed="0" sab_speed="0"
local sab_queue="0" sab_queue="0"
local sab_disk="0" sab_disk="0"
if lxc-info -n sabnzbd 2>/dev/null | grep -q "RUNNING"; then if /usr/bin/lxc-info -n sabnzbd 2>/dev/null | grep -q "RUNNING"; then
sab_running="true" sab_running=1
local api_key=$(get_sab_api) api_key=$(get_sab_api)
if [ -n "$api_key" ]; then if [ -n "$api_key" ]; then
local sab_data=$(curl -s "http://$SAB_IP:$SAB_PORT/api?mode=queue&output=json&apikey=$api_key" 2>/dev/null) sab_data=$(/usr/bin/curl -s "http://$SAB_IP:$SAB_PORT/api?mode=queue&output=json&apikey=$api_key" 2>/dev/null)
if [ -n "$sab_data" ]; then if [ -n "$sab_data" ]; then
sab_speed=$(echo "$sab_data" | jsonfilter -e '@.queue.speed' 2>/dev/null || echo "0") sab_speed=$(echo "$sab_data" | jsonfilter -e '@.queue.speed' 2>/dev/null || echo "0")
sab_queue=$(echo "$sab_data" | jsonfilter -e '@.queue.noofslots' 2>/dev/null || echo "0") sab_queue=$(echo "$sab_data" | jsonfilter -e '@.queue.noofslots' 2>/dev/null || echo "0")
sab_disk=$(echo "$sab_data" | jsonfilter -e '@.queue.diskspace1' 2>/dev/null || echo "0") sab_disk=$(echo "$sab_data" | jsonfilter -e '@.queue.diskspace1' 2>/dev/null || echo "0")
fi fi
fi fi
fi fi
# NZBHydra status # NZBHydra status
local hydra_running="false" hydra_running=0
if lxc-info -n nzbhydra 2>/dev/null | grep -q "RUNNING"; then if /usr/bin/lxc-info -n nzbhydra 2>/dev/null | grep -q "RUNNING"; then
hydra_running="true" hydra_running=1
fi fi
json_add_object "sabnzbd" json_add_object "sabnzbd"
json_add_boolean "running" "$sab_running" json_add_boolean "running" "$sab_running"
json_add_string "speed" "$sab_speed" json_add_string "speed" "$sab_speed"
json_add_int "queue_size" "$sab_queue" json_add_int "queue_size" "$sab_queue"
json_add_string "disk_free" "$sab_disk" json_add_string "disk_free" "$sab_disk"
json_add_string "url" "http://$SAB_IP:$SAB_PORT/" json_add_string "url" "http://$SAB_IP:$SAB_PORT/"
json_close_object json_close_object
json_add_object "nzbhydra" json_add_object "nzbhydra"
json_add_boolean "running" "$hydra_running" json_add_boolean "running" "$hydra_running"
json_add_string "url" "http://$HYDRA_IP:$HYDRA_PORT/" json_add_string "url" "http://$HYDRA_IP:$HYDRA_PORT/"
json_close_object json_close_object
json_dump json_dump
;; }
queue) do_queue() {
json_init json_init
json_add_array "items" json_add_array "items"
local api_key=$(get_sab_api) api_key=$(get_sab_api)
if [ -n "$api_key" ]; then if [ -n "$api_key" ]; then
local queue_data=$(curl -s "http://$SAB_IP:$SAB_PORT/api?mode=queue&output=json&apikey=$api_key" 2>/dev/null) queue_data=$(/usr/bin/curl -s "http://$SAB_IP:$SAB_PORT/api?mode=queue&output=json&apikey=$api_key" 2>/dev/null)
if [ -n "$queue_data" ]; then if [ -n "$queue_data" ]; then
echo "$queue_data" | python3 -c " echo "$queue_data" | python3 -c "
import sys, json import sys, json
try: try:
data = json.load(sys.stdin) data = json.load(sys.stdin)
@ -87,40 +81,37 @@ try:
except: except:
pass pass
" 2>/dev/null | while read item; do " 2>/dev/null | while read item; do
echo "$item" | { json_add_object ""
read line nzo_id=$(echo "$item" | jsonfilter -e '@.nzo_id')
json_add_object "" filename=$(echo "$item" | jsonfilter -e '@.filename')
local nzo_id=$(echo "$line" | jsonfilter -e '@.nzo_id') size=$(echo "$item" | jsonfilter -e '@.size')
local filename=$(echo "$line" | jsonfilter -e '@.filename') percentage=$(echo "$item" | jsonfilter -e '@.percentage')
local size=$(echo "$line" | jsonfilter -e '@.size') status=$(echo "$item" | jsonfilter -e '@.status')
local percentage=$(echo "$line" | jsonfilter -e '@.percentage') timeleft=$(echo "$item" | jsonfilter -e '@.timeleft')
local status=$(echo "$line" | jsonfilter -e '@.status') json_add_string "nzo_id" "$nzo_id"
local timeleft=$(echo "$line" | jsonfilter -e '@.timeleft') json_add_string "filename" "$filename"
json_add_string "nzo_id" "$nzo_id" json_add_string "size" "$size"
json_add_string "filename" "$filename" json_add_string "percentage" "$percentage"
json_add_string "size" "$size" json_add_string "status" "$status"
json_add_string "percentage" "$percentage" json_add_string "timeleft" "$timeleft"
json_add_string "status" "$status" json_close_object
json_add_string "timeleft" "$timeleft" done
json_close_object fi
} fi
done
fi
fi
json_close_array json_close_array
json_dump json_dump
;; }
history) do_history() {
json_init json_init
json_add_array "items" json_add_array "items"
local api_key=$(get_sab_api) api_key=$(get_sab_api)
if [ -n "$api_key" ]; then if [ -n "$api_key" ]; then
local hist_data=$(curl -s "http://$SAB_IP:$SAB_PORT/api?mode=history&limit=20&output=json&apikey=$api_key" 2>/dev/null) hist_data=$(/usr/bin/curl -s "http://$SAB_IP:$SAB_PORT/api?mode=history&limit=20&output=json&apikey=$api_key" 2>/dev/null)
if [ -n "$hist_data" ]; then if [ -n "$hist_data" ]; then
echo "$hist_data" | python3 -c " echo "$hist_data" | python3 -c "
import sys, json import sys, json
try: try:
data = json.load(sys.stdin) data = json.load(sys.stdin)
@ -135,41 +126,38 @@ try:
except: except:
pass pass
" 2>/dev/null | while read item; do " 2>/dev/null | while read item; do
echo "$item" | { json_add_object ""
read line nzo_id=$(echo "$item" | jsonfilter -e '@.nzo_id')
json_add_object "" name=$(echo "$item" | jsonfilter -e '@.name')
local nzo_id=$(echo "$line" | jsonfilter -e '@.nzo_id') size=$(echo "$item" | jsonfilter -e '@.size')
local name=$(echo "$line" | jsonfilter -e '@.name') status=$(echo "$item" | jsonfilter -e '@.status')
local size=$(echo "$line" | jsonfilter -e '@.size') json_add_string "nzo_id" "$nzo_id"
local status=$(echo "$line" | jsonfilter -e '@.status') json_add_string "name" "$name"
json_add_string "nzo_id" "$nzo_id" json_add_string "size" "$size"
json_add_string "name" "$name" json_add_string "status" "$status"
json_add_string "size" "$size" json_close_object
json_add_string "status" "$status" done
json_close_object fi
} fi
done
fi
fi
json_close_array json_close_array
json_dump json_dump
;; }
search) do_search() {
read -r input read -r input
json_load "$input" json_load "$input"
json_get_var query query json_get_var query query
json_init json_init
json_add_array "results" json_add_array "results"
if [ -n "$query" ]; then if [ -n "$query" ]; then
local search_url="http://$HYDRA_IP:$HYDRA_PORT/api?t=search&q=$(echo "$query" | sed 's/ /%20/g')" search_url="http://$HYDRA_IP:$HYDRA_PORT/api?t=search&q=$(echo "$query" | sed 's/ /%20/g')"
local result=$(curl -s "$search_url" 2>/dev/null) result=$(/usr/bin/curl -s "$search_url" 2>/dev/null)
if [ -n "$result" ]; then if [ -n "$result" ]; then
echo "$result" | python3 -c " echo "$result" | python3 -c "
import sys, xml.etree.ElementTree as ET, json import sys, xml.etree.ElementTree as ET, json
xml = sys.stdin.read() xml = sys.stdin.read()
try: try:
@ -182,106 +170,116 @@ try:
except: except:
pass pass
" 2>/dev/null | while read item; do " 2>/dev/null | while read item; do
echo "$item" | { json_add_object ""
read line title=$(echo "$item" | jsonfilter -e '@.title')
json_add_object "" link=$(echo "$item" | jsonfilter -e '@.link')
local title=$(echo "$line" | jsonfilter -e '@.title') size=$(echo "$item" | jsonfilter -e '@.size')
local link=$(echo "$line" | jsonfilter -e '@.link') json_add_string "title" "$title"
local size=$(echo "$line" | jsonfilter -e '@.size') json_add_string "link" "$link"
json_add_string "title" "$title" json_add_int "size" "$size"
json_add_string "link" "$link" json_close_object
json_add_int "size" "$size" done
json_close_object fi
} fi
done
fi
fi
json_close_array json_close_array
json_dump json_dump
;; }
add_nzb) do_add_nzb() {
read -r input read -r input
json_load "$input" json_load "$input"
json_get_var url url json_get_var url url
json_get_var category category json_get_var category category
json_init json_init
if [ -z "$url" ]; then if [ -z "$url" ]; then
json_add_boolean "success" 0 json_add_boolean "success" 0
json_add_string "error" "URL required" json_add_string "error" "URL required"
else else
local api_key=$(get_sab_api) api_key=$(get_sab_api)
local add_url="http://$SAB_IP:$SAB_PORT/api?mode=addurl&name=$(echo "$url" | sed 's/&/%26/g')&apikey=$api_key" add_url="http://$SAB_IP:$SAB_PORT/api?mode=addurl&name=$(echo "$url" | sed 's/&/%26/g')&apikey=$api_key"
[ -n "$category" ] && add_url="$add_url&cat=$category" [ -n "$category" ] && add_url="$add_url&cat=$category"
local result=$(curl -s "$add_url" 2>/dev/null) result=$(/usr/bin/curl -s "$add_url" 2>/dev/null)
if echo "$result" | grep -q "ok"; then if echo "$result" | grep -q "ok"; then
json_add_boolean "success" 1 json_add_boolean "success" 1
json_add_string "message" "NZB added to queue" json_add_string "message" "NZB added to queue"
else else
json_add_boolean "success" 0 json_add_boolean "success" 0
json_add_string "error" "Failed to add NZB" json_add_string "error" "Failed to add NZB"
fi fi
fi fi
json_dump json_dump
;; }
pause) do_pause() {
local api_key=$(get_sab_api) api_key=$(get_sab_api)
curl -s "http://$SAB_IP:$SAB_PORT/api?mode=pause&apikey=$api_key" >/dev/null 2>&1 /usr/bin/curl -s "http://$SAB_IP:$SAB_PORT/api?mode=pause&apikey=$api_key" >/dev/null 2>&1
json_init json_init
json_add_boolean "success" 1 json_add_boolean "success" 1
json_dump json_dump
;; }
resume) do_resume() {
local api_key=$(get_sab_api) api_key=$(get_sab_api)
curl -s "http://$SAB_IP:$SAB_PORT/api?mode=resume&apikey=$api_key" >/dev/null 2>&1 /usr/bin/curl -s "http://$SAB_IP:$SAB_PORT/api?mode=resume&apikey=$api_key" >/dev/null 2>&1
json_init json_init
json_add_boolean "success" 1 json_add_boolean "success" 1
json_dump json_dump
;; }
remove) do_remove() {
read -r input read -r input
json_load "$input" json_load "$input"
json_get_var nzo_id nzo_id json_get_var nzo_id nzo_id
local api_key=$(get_sab_api) api_key=$(get_sab_api)
curl -s "http://$SAB_IP:$SAB_PORT/api?mode=queue&name=delete&value=$nzo_id&apikey=$api_key" >/dev/null 2>&1 /usr/bin/curl -s "http://$SAB_IP:$SAB_PORT/api?mode=queue&name=delete&value=$nzo_id&apikey=$api_key" >/dev/null 2>&1
json_init json_init
json_add_boolean "success" 1 json_add_boolean "success" 1
json_dump json_dump
;; }
config) do_config() {
json_init json_init
# NNTP servers # NNTP servers
json_add_array "servers" json_add_array "servers"
for section in $(uci show sabnzbd 2>/dev/null | grep "=nntp$" | sed 's/sabnzbd\.\(.*\)=nntp/\1/'); do for section in $(uci show sabnzbd 2>/dev/null | grep "=nntp$" | sed 's/sabnzbd\.\(.*\)=nntp/\1/'); do
json_add_object "" json_add_object ""
json_add_string "id" "$section" json_add_string "id" "$section"
json_add_string "name" "$(uci -q get sabnzbd.$section.name)" json_add_string "name" "$(uci -q get sabnzbd.$section.name)"
json_add_string "host" "$(uci -q get sabnzbd.$section.host)" json_add_string "host" "$(uci -q get sabnzbd.$section.host)"
json_add_string "port" "$(uci -q get sabnzbd.$section.port)" json_add_string "port" "$(uci -q get sabnzbd.$section.port)"
json_add_boolean "ssl" "$(uci -q get sabnzbd.$section.ssl)" json_add_boolean "ssl" "$(uci -q get sabnzbd.$section.ssl)"
json_add_string "connections" "$(uci -q get sabnzbd.$section.connections)" json_add_string "connections" "$(uci -q get sabnzbd.$section.connections)"
json_close_object json_close_object
done done
json_close_array json_close_array
json_dump json_dump
;; }
*) case "$1" in
echo '{"error":"Unknown method"}' list)
;; echo '{"status":{},"queue":{},"history":{},"search":{"query":"string"},"add_nzb":{"url":"string","category":"string"},"pause":{},"resume":{},"remove":{"nzo_id":"string"},"config":{}}'
;;
call)
case "$2" in
status) do_status ;;
queue) do_queue ;;
history) do_history ;;
search) do_search ;;
add_nzb) do_add_nzb ;;
pause) do_pause ;;
resume) do_resume ;;
remove) do_remove ;;
config) do_config ;;
*) echo '{"error":"Unknown method"}' ;;
esac esac
;; ;;
esac esac