Merge branch 'master' into issues/57-move-logger-call-in-subroutine

Author: devkapilbansal

openwisp-monitoring/files/monitoring.agent

@@ -66,7 +66,7 @@ time_to_seconds() {
 		|| { [ "${time%h}" -ge 1 ] 2>/dev/null && seconds=$((${time%h} * 3600)); } \
 		|| { [ "${time%d}" -ge 1 ] 2>/dev/null && seconds=$((${time%d} * 86400)); }
 
-	echo $seconds
+	echo "$seconds"
 	unset seconds
 	unset time
 }
@@ -100,7 +100,7 @@ collect_data() {
 	until [ "$n" -ge 5 ]; do
 		/usr/sbin/netjson-monitoring --dump "$MONITORED_INTERFACES" && break
 
-		[ "$n" -eq 5 ] && log -e -v "Collecting data failed!"
+		[ "$n" -eq 4 ] && log -e -v "Collecting data failed!"
 		n=$((n + 1))
 		sleep 5
 	done
@@ -201,6 +201,7 @@ send_data() {
 				# send data
 				response_code=$($CURL_COMMAND -H "Content-Type: application/json" -d "$data" "$url")
 				if [ "$response_code" = "200" ]; then
+					success=$((success + 1))
 					log -i -v "Data sent successfully."
 					# check if agent was already failing or not to avoid repeating log messages
 					if [ "$FAILING" -eq "1" ]; then