mirror of
https://github.com/EV21/dynb.git
synced 2025-12-26 16:39:32 +01:00
fix: incorrect status code handling
no persistent status file was written in case of an error as the return status code was always 0
This commit is contained in:
13
dynb.sh
13
dynb.sh
@@ -315,7 +315,7 @@ function send_request
|
|||||||
--user "$DYNB_USERNAME":"$DYNB_PASSWORD" \
|
--user "$DYNB_USERNAME":"$DYNB_PASSWORD" \
|
||||||
"${dyndns_update_url}")
|
"${dyndns_update_url}")
|
||||||
analyse_response
|
analyse_response
|
||||||
return $?
|
status_code=$?
|
||||||
;;
|
;;
|
||||||
deSEC* | desec* | dedyn*)
|
deSEC* | desec* | dedyn*)
|
||||||
_response=$(curl --silent "$_interface_str" \
|
_response=$(curl --silent "$_interface_str" \
|
||||||
@@ -324,7 +324,7 @@ function send_request
|
|||||||
--get --data-urlencode "hostname=$DYNB_DYN_DOMAIN" \
|
--get --data-urlencode "hostname=$DYNB_DYN_DOMAIN" \
|
||||||
"${dyndns_update_url}")
|
"${dyndns_update_url}")
|
||||||
analyse_response
|
analyse_response
|
||||||
return $?
|
status_code=$?
|
||||||
;;
|
;;
|
||||||
[Ii][Pp][Vv]64* )
|
[Ii][Pp][Vv]64* )
|
||||||
_response=$(curl --silent "$_interface_str" \
|
_response=$(curl --silent "$_interface_str" \
|
||||||
@@ -334,7 +334,7 @@ function send_request
|
|||||||
--form "domain=$DYNB_DYN_DOMAIN" \
|
--form "domain=$DYNB_DYN_DOMAIN" \
|
||||||
"${dyndns_update_url}")
|
"${dyndns_update_url}")
|
||||||
analyse_response
|
analyse_response
|
||||||
return $?
|
status_code=$?
|
||||||
;;
|
;;
|
||||||
dynv6* | duckDNS* | duckdns* | ddnss*)
|
dynv6* | duckDNS* | duckdns* | ddnss*)
|
||||||
_response=$(
|
_response=$(
|
||||||
@@ -342,9 +342,10 @@ function send_request
|
|||||||
--user-agent "$_userAgent" \
|
--user-agent "$_userAgent" \
|
||||||
"${dyndns_update_url}")
|
"${dyndns_update_url}")
|
||||||
analyse_response
|
analyse_response
|
||||||
return $?
|
status_code=$?
|
||||||
;;
|
;;
|
||||||
esac
|
esac
|
||||||
|
return $status_code
|
||||||
}
|
}
|
||||||
|
|
||||||
function analyse_response
|
function analyse_response
|
||||||
@@ -442,18 +443,20 @@ function dynupdate
|
|||||||
then
|
then
|
||||||
dyndns_update_url="${dyndns_update_url}${myip_str}=${_new_IPv4}&${myipv6_str}=${_new_IPv6}"
|
dyndns_update_url="${dyndns_update_url}${myip_str}=${_new_IPv4}&${myipv6_str}=${_new_IPv6}"
|
||||||
send_request
|
send_request
|
||||||
|
request_status=$?
|
||||||
fi
|
fi
|
||||||
if [[ $_is_IPv4_enabled == true ]] && [[ $_is_IPv6_enabled == false ]]
|
if [[ $_is_IPv4_enabled == true ]] && [[ $_is_IPv6_enabled == false ]]
|
||||||
then
|
then
|
||||||
dyndns_update_url="${dyndns_update_url}${myip_str}=${_new_IPv4}"
|
dyndns_update_url="${dyndns_update_url}${myip_str}=${_new_IPv4}"
|
||||||
send_request
|
send_request
|
||||||
|
request_status=$?
|
||||||
fi
|
fi
|
||||||
if [[ $_is_IPv4_enabled == false ]] && [[ $_is_IPv6_enabled == true ]]
|
if [[ $_is_IPv4_enabled == false ]] && [[ $_is_IPv6_enabled == true ]]
|
||||||
then
|
then
|
||||||
dyndns_update_url="${dyndns_update_url}${myipv6_str}=${_new_IPv6}"
|
dyndns_update_url="${dyndns_update_url}${myipv6_str}=${_new_IPv6}"
|
||||||
send_request
|
send_request
|
||||||
|
request_status=$?
|
||||||
fi
|
fi
|
||||||
request_status=$?
|
|
||||||
debugMessage "Update URL was: $dyndns_update_url"
|
debugMessage "Update URL was: $dyndns_update_url"
|
||||||
return $request_status
|
return $request_status
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user