mirror of
https://github.com/wled/WLED.git
synced 2025-07-17 07:46:32 +00:00
Implement error checking for curl; add checks for required param args
This commit is contained in:
parent
dc5eaf3ae9
commit
646ec30ae5
@ -104,11 +104,22 @@ backup_one() {
|
|||||||
local curl_command_cfg="curl -s "$cfg_url" -o "$cfg_dest.tmp""
|
local curl_command_cfg="curl -s "$cfg_url" -o "$cfg_dest.tmp""
|
||||||
local curl_command_presets="curl -s "$presets_url" -o "$presets_dest.tmp""
|
local curl_command_presets="curl -s "$presets_url" -o "$presets_dest.tmp""
|
||||||
|
|
||||||
curl_handler "$curl_command_cfg" "$hostname"
|
if ! curl_handler "$curl_command_cfg" "$hostname"; then
|
||||||
curl_handler "$curl_command_presets" "$hostname"
|
log "ERROR" "$RED" "Failed to backup configuration for $hostname"
|
||||||
|
rm -f "$cfg_dest.tmp"
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
|
||||||
|
if ! curl_handler "$curl_command_presets" "$hostname"; then
|
||||||
|
log "ERROR" "$RED" "Failed to backup presets for $hostname"
|
||||||
|
rm -f "$presets_dest.tmp"
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
|
||||||
mv "$cfg_dest.tmp" "$cfg_dest"
|
mv "$cfg_dest.tmp" "$cfg_dest"
|
||||||
mv "$presets_dest.tmp" "$presets_dest"
|
mv "$presets_dest.tmp" "$presets_dest"
|
||||||
|
log "INFO" "$GREEN" "Successfully backed up config and presets for $hostname"
|
||||||
|
return 0
|
||||||
}
|
}
|
||||||
|
|
||||||
# Update one device
|
# Update one device
|
||||||
@ -146,6 +157,10 @@ while [[ $# -gt 0 ]]; do
|
|||||||
exit 0
|
exit 0
|
||||||
;;
|
;;
|
||||||
-t|--target)
|
-t|--target)
|
||||||
|
if [ -z "$2" ] || [[ "$2" == -* ]]; then
|
||||||
|
log "ERROR" "$RED" "The --target option requires an argument."
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
target="$2"
|
target="$2"
|
||||||
shift 2
|
shift 2
|
||||||
;;
|
;;
|
||||||
@ -154,10 +169,18 @@ while [[ $# -gt 0 ]]; do
|
|||||||
shift
|
shift
|
||||||
;;
|
;;
|
||||||
-d|--directory)
|
-d|--directory)
|
||||||
|
if [ -z "$2" ] || [[ "$2" == -* ]]; then
|
||||||
|
log "ERROR" "$RED" "The --directory option requires an argument."
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
backup_dir="$2"
|
backup_dir="$2"
|
||||||
shift 2
|
shift 2
|
||||||
;;
|
;;
|
||||||
-f|--firmware)
|
-f|--firmware)
|
||||||
|
if [ -z "$2" ] || [[ "$2" == -* ]]; then
|
||||||
|
log "ERROR" "$RED" "The --firmware option requires an argument."
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
firmware_file="$2"
|
firmware_file="$2"
|
||||||
shift 2
|
shift 2
|
||||||
;;
|
;;
|
||||||
@ -174,7 +197,6 @@ while [[ $# -gt 0 ]]; do
|
|||||||
exit 1
|
exit 1
|
||||||
;;
|
;;
|
||||||
esac
|
esac
|
||||||
|
|
||||||
done
|
done
|
||||||
|
|
||||||
# Execute the appropriate command
|
# Execute the appropriate command
|
||||||
|
Loading…
x
Reference in New Issue
Block a user