From f59c21e72e012dcc659a49bda38765589a7cabe0 Mon Sep 17 00:00:00 2001
From: Alexandre <44178713+alexbelgium@users.noreply.github.com>
Date: Mon, 24 Nov 2025 08:13:46 +0100
Subject: [PATCH] Refactor public_ip.sh for better portability and error
 handling

Updated shebang to use env for portability and improved error handling in public IP retrieval.
---
 qbittorrent/rootfs/etc/helpers/public_ip.sh | 23 ++++++++++++++-------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/qbittorrent/rootfs/etc/helpers/public_ip.sh b/qbittorrent/rootfs/etc/helpers/public_ip.sh
index 262e04823..c0bdc5d44 100644
--- a/qbittorrent/rootfs/etc/helpers/public_ip.sh
+++ b/qbittorrent/rootfs/etc/helpers/public_ip.sh
@@ -1,14 +1,20 @@
-#!/bin/bash
+#!/usr/bin/env bash
+# get-public-ip: query several IP services in random order and print the first result
+
+set -euo pipefail
 
 get_public_ip() {
-    local urls=(
+    local -a urls=(
         "https://ifconfig.co/ip"
         "https://api64.ipify.org"
         "https://ipecho.net/plain"
     )
-    local i j tmp url ip
+    local ip=""
+    local i=0
+    local j=0
+    local tmp=""
 
-    # Fisher–Yates shuffle to randomize the order
+    # Fisher–Yates shuffle to randomize order
     for ((i=${#urls[@]}-1; i>0; i--)); do
         j=$((RANDOM % (i + 1)))
         tmp=${urls[i]}
@@ -18,7 +24,8 @@ get_public_ip() {
 
     # Try each provider in random order until one works
     for url in "${urls[@]}"; do
-        if ip=$(curl -fsS --max-time 10 "$url"); then
+        # Suppress curl error messages
+        if ip=$(curl -fsS --max-time 10 "$url" 2>/dev/null); then
             printf '%s\n' "$ip"
             return 0
         fi
@@ -27,7 +34,7 @@ get_public_ip() {
     return 1
 }
 
-# Write to /currentip, fail if none of the services respond
-if ! get_public_ip > /currentip; then
-    echo "Failed to get public IP from all providers" >&2
+if ! get_public_ip; then
+    echo "get-public-ip: failed to get public IP from all providers" >&2
+    exit 1
 fi