Merge pull request #3683 from chris5560/ddns-scripts

ddns-scripts: fix return codes if provider's data already up to date

net/ddns-scripts/Makefile

@@ -12,7 +12,7 @@ PKG_NAME:=ddns-scripts
 PKG_VERSION:=2.7.6
 # Release == build
 # increase on changes of services files or tld_names.dat
-PKG_RELEASE:=6
+PKG_RELEASE:=7
 
 PKG_LICENSE:=GPL-2.0
 PKG_MAINTAINER:=Christian Schoenebeck <christian.schoenebeck@gmail.com>

net/ddns-scripts/files/update_cloudflare_com_v4.sh

@@ -124,7 +124,7 @@ cloudflare_transfer || return 1
 __ZONEID=$(grep -o '"id":"[^"]*' $DATFILE | grep -o '[^"]*$' | head -1)
 [ -z "$__ZONEID" ] && {
 	write_log 4 "Could not detect 'zone id' for domain.tld: '$__DOMAIN'"
-	return 1
+	return 127
 }
 
 # read record id for A or AAAA record of host.domain.TLD
@@ -134,7 +134,7 @@ cloudflare_transfer || return 1
 __RECID=$(grep -o '"id":"[^"]*' $DATFILE | grep -o '[^"]*$' | head -1)
 [ -z "$__RECID" ] && {
 	write_log 4 "Could not detect 'record id' for host.domain.tld: '$__HOST'"
-	return 1
+	return 127
 }
 
 # extract current stored IP
@@ -153,12 +153,12 @@ __DATA=$(grep -o '"content":"[^"]*' $DATFILE | grep -o '[^"]*$' | head -1)
 		expand_ipv6 $__DATA __DATA
 		[ "$__DATA" = "$__IPV6" ] && {		# IPv6 no update needed
 			write_log 7 "IPv6 at CloudFlare.com already up to date"
-			return 127
+			return 0
 		}
 	else
 		[ "$__DATA" = "$__IP" ] && {		# IPv4 no update needed
 			write_log 7 "IPv4 at CloudFlare.com already up to date"
-			return 127
+			return 0
 		}
 	fi
 }

net/ddns-scripts/files/update_godaddy_com_v1.sh

@@ -136,12 +136,12 @@ __DATA=$(sed -r 's/.+data":"(.+)","t.+/\1/g' $DATFILE)
 		expand_ipv6 $__DATA __DATA
 		[ "$__DATA" = "$__IPV6" ] && {		# IPv6 no update needed
 			write_log 7 "IPv6 at GoDaddy.com already up to date"
-			return 127
+			return 0
 		}
 	else
 		[ "$__DATA" = "$__IP" ] && {		# IPv4 no update needed
 			write_log 7 "IPv4 at GoDaddy.com already up to date"
-			return 127
+			return 0
 		}
 	fi
 }