ddns-scripts: rename variable: s/ERR_UPDATE/RETRY_COUNT/
authorRafał Miłecki <rafal@milecki.pl>
Tue, 27 Sep 2022 15:42:58 +0000 (17:42 +0200)
committerRafał Miłecki <rafal@milecki.pl>
Tue, 27 Sep 2022 15:46:04 +0000 (17:46 +0200)
Rename variable to make code easier to understand. This variable
specifies how many times in row ddns script tried to update IP without a
success.

Previous name ("ERR_UPDATE") didn't suggest it was for counting
anything. It also didn't specify was error was it related to.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
net/ddns-scripts/files/usr/lib/ddns/dynamic_dns_functions.sh
net/ddns-scripts/files/usr/lib/ddns/dynamic_dns_updater.sh

index af2ef32aedaf078b3642242da2f8a8e37d5d32bd..ea1b2a7acf798c49f78da122d3746a45f0f4846b 100644 (file)
@@ -57,7 +57,7 @@ URL_PENC=""           # url encoded $param_enc from config file
 UPD_ANSWER=""          # Answer given by service on success
 
 ERR_LAST=0             # used to save $? return code of program and function calls
-ERR_UPDATE=0           # error counter on different current and registered IPs
+RETRY_COUNT=0          # error counter on different current and registered IPs
 
 PID_SLEEP=0            # ProcessID of current background "sleep"
 
index 210d7339b05f53241f91d2364cba8c461a16cd5e..a51ad511423d01f7754bd1c20f2c7d2fe009e181 100644 (file)
@@ -411,10 +411,10 @@ while : ; do
        # IP's are still different
        if [ "$CURRENT_IP" != "$REGISTERED_IP" ]; then
                if [ $VERBOSE -le 1 ]; then     # VERBOSE <=1 then retry
-                       ERR_UPDATE=$(( $ERR_UPDATE + 1 ))
-                       [ $retry_max_count -gt 0 -a $ERR_UPDATE -gt $retry_max_count ] && \
+                       RETRY_COUNT=$(( $RETRY_COUNT + 1 ))
+                       [ $retry_max_count -gt 0 -a $RETRY_COUNT -gt $retry_max_count ] && \
                                write_log 14 "Updating IP at DDNS provider failed after $retry_max_count retries"
-                       write_log 4 "Updating IP at DDNS provider failed - starting retry $ERR_UPDATE/$retry_max_count"
+                       write_log 4 "Updating IP at DDNS provider failed - starting retry $RETRY_COUNT/$retry_max_count"
                        continue # loop to beginning
                else
                        write_log 4 "Updating IP at DDNS provider failed"
@@ -422,7 +422,7 @@ while : ; do
                fi
        else
                # we checked successful the last update
-               ERR_UPDATE=0                    # reset error counter
+               RETRY_COUNT=0                   # reset error counter
        fi
 
        # force_update=0 or VERBOSE > 1 - leave here