diff --git a/README.md b/README.md index 8f1f105..06be3f4 100644 --- a/README.md +++ b/README.md @@ -184,7 +184,7 @@ To run the backup automatically, edit the root crontab. ```ini # ================================================================= -# Configuration for rsync Backup Script v0.32 +# Configuration for rsync Backup Script v0.33 # ================================================================= # !! IMPORTANT !! Set file permissions to 600 (chmod 600 backup.conf) @@ -310,7 +310,7 @@ END_EXCLUDES ```bash #!/bin/bash -# ===================== v0.32 - 2025.08.13 ======================== +# ===================== v0.33 - 2025.08.14 ======================== # # ================================================================= # SCRIPT INITIALIZATION & SETUP @@ -617,6 +617,7 @@ run_preflight_checks() { fi } run_restore_mode() { + local RSYNC_CMD="rsync" printf "${C_BOLD}${C_CYAN}--- RESTORE MODE ACTIVATED ---${C_RESET}\n" run_preflight_checks "restore" local DIRS_ARRAY; read -ra DIRS_ARRAY <<< "$BACKUP_DIRS" @@ -639,11 +640,9 @@ run_restore_mode() { if [[ "$dir_choice" == "$RECYCLE_OPTION" ]]; then printf "${C_BOLD}${C_CYAN}--- Browse Recycle Bin ---${C_RESET}\n" local remote_recycle_path="${BOX_DIR%/}/${RECYCLE_BIN_DIR%/}" - local date_folders - date_folders=$(ssh "${SSH_OPTS_ARRAY[@]}" "${SSH_DIRECT_OPTS[@]}" "$BOX_ADDR" "ls -1 \"$remote_recycle_path\"" 2>/dev/null) || true + local date_folders; date_folders=$(ssh "${SSH_OPTS_ARRAY[@]}" "${SSH_DIRECT_OPTS[@]}" "$BOX_ADDR" "ls -1 \"$remote_recycle_path\"" 2>/dev/null) || true if [[ -z "$date_folders" ]]; then - echo "❌ No dated folders found in the recycle bin. Nothing to restore." >&2 - return 1 + echo "❌ No dated folders found in the recycle bin. Nothing to restore." >&2; return 1 fi printf "${C_YELLOW}Select a backup run (date_time) to browse:${C_RESET}\n" select date_choice in $date_folders "Cancel"; do @@ -654,61 +653,113 @@ run_restore_mode() { local remote_date_path="${remote_recycle_path}/${date_choice}" printf "${C_BOLD}--- Files available from ${date_choice} (showing first 20) ---${C_RESET}\n" local remote_listing_source="${BOX_ADDR}:${remote_date_path}/" - rsync -r -n --out-format='%n' -e "$SSH_CMD" "$remote_listing_source" . 2>/dev/null | head -n 20 || echo "No files found for this date." + "$RSYNC_CMD" -r -n --out-format='%n' -e "$SSH_CMD" "$remote_listing_source" . 2>/dev/null | head -n 20 || echo "No files found for this date." printf "${C_BOLD}--------------------------------------------------------${C_RESET}\n" - printf "${C_YELLOW}Enter the full original path of the item to restore (e.g., home/user/file.txt): ${C_RESET}" - read -r specific_path + printf "${C_YELLOW}Enter the full original path of the item to restore (e.g., home/user/file.txt): ${C_RESET}"; read -r specific_path specific_path=$(echo "$specific_path" | sed 's#^/##') if [[ -z "$specific_path" ]]; then echo "❌ Path cannot be empty. Aborting."; return 1; fi full_remote_source="${BOX_ADDR}:${remote_date_path}/${specific_path}" - if ! rsync -r -n -e "$SSH_CMD" "$full_remote_source" . >/dev/null 2>&1; then - echo "❌ ERROR: The path '${specific_path}' was not found in the recycle bin for ${date_choice}. Aborting." >&2 - return 1 + if ! "$RSYNC_CMD" -r -n -e "$SSH_CMD" "$full_remote_source" . >/dev/null 2>&1; then + echo "❌ ERROR: The path '${specific_path}' was not found in the recycle bin for ${date_choice}. Aborting." >&2; return 1 fi - default_local_dest="/${specific_path}" - item_for_display="(from Recycle Bin) '${specific_path}'" + default_local_dest="/${specific_path}"; item_for_display="(from Recycle Bin) '${specific_path}'" elif [[ "$dir_choice" == "Cancel" ]]; then - echo "Restore cancelled." - return 0 + echo "Restore cancelled."; return 0 else item_for_display="the entire directory '${dir_choice}'" while true; do - printf "\n${C_YELLOW}Restore the entire directory or a specific file/subfolder? [entire/specific]: ${C_RESET}" - read -r choice + printf "\n${C_YELLOW}Restore the entire directory or a specific file/subfolder? [entire/specific]: ${C_RESET}"; read -r choice case "$choice" in - entire) - is_full_directory_restore=true - break - ;; + entire) is_full_directory_restore=true; break ;; specific) - local specific_path_prompt - printf -v specific_path_prompt "Enter the path relative to '%s' to restore: " "$dir_choice" - printf "${C_YELLOW}%s${C_RESET}" "$specific_path_prompt" - read -er specific_path + printf -v specific_path_prompt "Enter the path relative to '%s' to restore: " "$dir_choice"; printf "${C_YELLOW}%s${C_RESET}" "$specific_path_prompt"; read -er specific_path specific_path=$(echo "$specific_path" | sed 's#^/##') if [[ -n "$specific_path" ]]; then - restore_path="$specific_path" - item_for_display="'$restore_path' from '${dir_choice}'" - break + restore_path="$specific_path"; item_for_display="'$restore_path' from '${dir_choice}'"; break else echo "Path cannot be empty. Please try again or choose 'entire'." - fi - ;; + fi ;; *) echo "Invalid choice. Please answer 'entire' or 'specific'." ;; esac done local relative_path="${dir_choice#*./}" - full_remote_source="${REMOTE_TARGET}${relative_path}${restore_path}" + local remote_base="${REMOTE_TARGET%/}" + full_remote_source="${remote_base}/${relative_path#/}" + if [[ -n "$restore_path" ]]; then + full_remote_source="${full_remote_source%/}/${restore_path#/}" + fi if [[ -n "$restore_path" ]]; then default_local_dest=$(echo "${dir_choice}${restore_path}" | sed 's#/\./#/#') else default_local_dest=$(echo "$dir_choice" | sed 's#/\./#/#') fi fi - local final_dest - printf "\n${C_YELLOW}Enter the destination path.\n${C_DIM}Press [Enter] to use the original location (%s):${C_RESET} " "$default_local_dest" - read -r final_dest + local final_dest + printf "\n%s\n" "${C_BOLD}--------------------------------------------------------" + printf "%s\n" " Restore Destination" + printf "%s\n" "--------------------------------------------------------${C_RESET}" + printf "%s\n\n" "Enter the absolute destination path for the restore." + printf "%s\n" "${C_YELLOW}Default (original location):${C_RESET}" + printf "${C_CYAN}%s${C_RESET}\n\n" "$default_local_dest" + printf "%s\n" "Press [Enter] to use the default path, or enter a new one." + read -rp "> " final_dest : "${final_dest:=$default_local_dest}" + local path_validation_attempts=0 + local max_attempts=5 + while true; do + ((path_validation_attempts++)) + if (( path_validation_attempts > max_attempts )); then + printf "\n${C_RED}❌ Too many invalid attempts. Exiting restore mode.${C_RESET}\n"; return 1 + fi + if [[ "$final_dest" != "/" ]]; then final_dest="${final_dest%/}"; fi + local parent_dir; parent_dir=$(dirname -- "$final_dest") + if [[ "$final_dest" != /* ]]; then + printf "\n${C_RED}❌ Error: Please provide an absolute path (starting with '/').${C_RESET}\n" + elif [[ -e "$final_dest" && ! -d "$final_dest" ]]; then + printf "\n${C_RED}❌ Error: The destination '%s' exists but is a file. Please choose a different path.${C_RESET}\n" "$final_dest" + elif [[ -e "$parent_dir" && ! -w "$parent_dir" ]]; then + printf "\n${C_RED}❌ Error: The parent directory '%s' exists but is not writable.${C_RESET}\n" "$parent_dir" + elif [[ -d "$final_dest" ]]; then + printf "${C_GREEN}✅ Destination '%s' exists and is accessible.${C_RESET}\n" "$final_dest" + if [[ "$final_dest" != "$default_local_dest" && -z "$restore_path" ]]; then + local warning_msg="⚠️ WARNING: Custom destination directory already exists. Files may be overwritten." + printf "${C_YELLOW}%s${C_RESET}\n" "$warning_msg"; log_message "$warning_msg" + fi + break + else + printf "\n${C_YELLOW}⚠️ The destination '%s' does not exist.${C_RESET}\n" "$final_dest" + printf "${C_YELLOW}Choose an action:${C_RESET}\n" + PS3="Your choice: " + select action in "Create the destination path" "Enter a different path" "Cancel"; do + case "$action" in + "Create the destination path") + if mkdir -p "$final_dest"; then + printf "${C_GREEN}✅ Successfully created directory '%s'.${C_RESET}\n" "$final_dest" + if [[ "${is_full_directory_restore:-false}" == "true" ]]; then + chmod 700 "$final_dest"; log_message "Set permissions to 700 on newly created restore directory: $final_dest" + else + chmod 755 "$final_dest" + fi + break 2 + else + printf "\n${C_RED}❌ Failed to create directory '%s'. Check permissions.${C_RESET}\n" "$final_dest"; break + fi ;; + "Enter a different path") + break ;; + "Cancel") + echo "Restore cancelled by user."; return 0 ;; + *) echo "Invalid option. Please try again." ;; + esac + done + PS3="#? " + fi + if (( path_validation_attempts < max_attempts )); then + printf "\n${C_YELLOW}Please enter a new destination path: ${C_RESET}"; read -r final_dest + if [[ -z "$final_dest" ]]; then + final_dest="$default_local_dest"; printf "${C_DIM}Empty input, using default location: %s${C_RESET}\n" "$final_dest" + fi + fi + done local extra_rsync_opts=() local dest_user="" if [[ "$final_dest" == /home/* ]]; then @@ -716,60 +767,41 @@ run_restore_mode() { if [[ -n "$dest_user" ]] && id -u "$dest_user" &>/dev/null; then printf "${C_CYAN}ℹ️ Home directory detected. Restored files will be owned by '${dest_user}'.${C_RESET}\n" extra_rsync_opts+=("--chown=${dest_user}:${dest_user}") + chown "${dest_user}:${dest_user}" "$final_dest" 2>/dev/null || true else dest_user="" fi fi - local dest_created=false - if [[ ! -e "$final_dest" ]]; then - dest_created=true - fi - local dest_parent - dest_parent=$(dirname "$final_dest") - if ! mkdir -p "$dest_parent"; then - echo "❌ FATAL: Could not create parent destination directory '$dest_parent'. Aborting." >&2 - return 1 - fi - if [[ -n "$dest_user" ]]; then - chown "${dest_user}:${dest_user}" "$dest_parent" - fi - if [[ "$final_dest" != "$default_local_dest" && -d "$final_dest" && -z "$restore_path" ]]; then - local warning_msg="⚠️ WARNING: The custom destination directory '$final_dest' already exists. Files may be overwritten." - echo "$warning_msg"; log_message "$warning_msg" - fi - if [[ "$dest_created" == "true" && "${is_full_directory_restore:-false}" == "true" ]]; then - chmod 700 "$final_dest"; log_message "Set permissions to 700 on newly created restore directory: $final_dest" - fi - printf "Restore destination is set to: ${C_BOLD}%s${C_RESET}\n" "$final_dest" - printf "\n${C_BOLD}${C_YELLOW}--- PERFORMING DRY RUN. NO FILES WILL BE CHANGED. ---${C_RESET}\n" + printf "\n${C_BOLD}Restore Summary:${C_RESET}\n" + printf " Source: %s\n" "$item_for_display" + printf " Destination: ${C_BOLD}%s${C_RESET}\n" "$final_dest" + printf "\n${C_BOLD}${C_YELLOW}--- PERFORMING DRY RUN (NO CHANGES MADE) ---${C_RESET}\n" log_message "Starting restore dry-run of ${item_for_display} from ${full_remote_source} to ${final_dest}" local rsync_restore_opts=(-avhi --progress --exclude-from="$EXCLUDE_FILE_TMP" -e "$SSH_CMD") - if ! rsync "${rsync_restore_opts[@]}" "${extra_rsync_opts[@]}" --dry-run "$full_remote_source" "$final_dest"; then - echo "❌ DRY RUN FAILED. Rsync reported an error. Aborting." >&2; return 1 + if ! "$RSYNC_CMD" "${rsync_restore_opts[@]}" "${extra_rsync_opts[@]}" --dry-run "$full_remote_source" "$final_dest"; then + printf "${C_RED}❌ DRY RUN FAILED. Rsync reported an error. Check connectivity and permissions.${C_RESET}\n" >&2 + log_message "Restore dry-run failed for ${item_for_display}"; return 1 fi printf "${C_BOLD}${C_GREEN}--- DRY RUN COMPLETE ---${C_RESET}\n" - local confirmation while true; do - printf "\n${C_YELLOW}Are you sure you want to proceed with restoring %s to '%s'? [yes/no]: ${C_RESET}" "$item_for_display" "$final_dest" - read -r confirmation - - case "$confirmation" in - yes) break ;; - no) echo "Restore aborted by user." ; return 0 ;; - *) echo "Please answer yes or no." ;; + printf "\n${C_YELLOW}Proceed with restoring %s to '%s'? [yes/no]: ${C_RESET}" "$item_for_display" "$final_dest"; read -r confirmation + case "${confirmation,,}" in + yes|y) break ;; + no|n) echo "Restore cancelled by user."; return 0 ;; + *) echo "Please answer 'yes' or 'no'." ;; esac done - printf "\n${C_BOLD}--- PROCEEDING WITH RESTORE... ---${C_RESET}\n" - log_message "Starting REAL restore of ${item_for_display} from ${full_remote_source} to ${final_dest}" - if rsync "${rsync_restore_opts[@]}" "${extra_rsync_opts[@]}" "$full_remote_source" "$final_dest"; then + printf "\n${C_BOLD}--- EXECUTING RESTORE ---${C_RESET}\n" + log_message "Starting actual restore of ${item_for_display} from ${full_remote_source} to ${final_dest}" + if "$RSYNC_CMD" "${rsync_restore_opts[@]}" "${extra_rsync_opts[@]}" "$full_remote_source" "$final_dest"; then log_message "Restore completed successfully." printf "${C_GREEN}✅ Restore of %s to '%s' completed successfully.${C_RESET}\n" "$item_for_display" "$final_dest" send_notification "Restore SUCCESS: ${HOSTNAME}" "white_check_mark" "${NTFY_PRIORITY_SUCCESS}" "success" "Successfully restored ${item_for_display} to ${final_dest}" else - log_message "Restore FAILED with rsync exit code $?." + local rsync_exit_code=$? + log_message "Restore FAILED with rsync exit code ${rsync_exit_code}." printf "${C_RED}❌ Restore FAILED. Check the rsync output and log for details.${C_RESET}\n" - send_notification "Restore FAILED: ${HOSTNAME}" "x" "${NTFY_PRIORITY_FAILURE}" "failure" "Restore of ${item_for_display} to ${final_dest} failed." - return 1 + send_notification "Restore FAILED: ${HOSTNAME}" "x" "${NTFY_PRIORITY_FAILURE}" "failure" "Restore of ${item_for_display} to ${final_dest} failed (exit code: ${rsync_exit_code})"; return 1 fi } run_recycle_bin_cleanup() { diff --git a/backup.conf b/backup.conf index 46c07ad..0b95fc0 100644 --- a/backup.conf +++ b/backup.conf @@ -1,5 +1,5 @@ # ================================================================= -# Configuration for rsync Backup Script v0.32 +# Configuration for rsync Backup Script v0.33 # ================================================================= # !! IMPORTANT !! Set file permissions to 600 (chmod 600 backup.conf) diff --git a/backup_script.sh b/backup_script.sh index 1cf56c8..ff17884 100644 --- a/backup_script.sh +++ b/backup_script.sh @@ -1,5 +1,5 @@ #!/bin/bash -# ===================== v0.32 - 2025.08.13 ======================== +# ===================== v0.33 - 2025.08.14 ======================== # # Example backup.conf: # BACKUP_DIRS="/home/user/test/./ /var/www/./" @@ -359,14 +359,19 @@ run_restore_mode() { if [[ "$dir_choice" == "$RECYCLE_OPTION" ]]; then printf "${C_BOLD}${C_CYAN}--- Browse Recycle Bin ---${C_RESET}\n" local remote_recycle_path="${BOX_DIR%/}/${RECYCLE_BIN_DIR%/}" - local date_folders - date_folders=$(ssh "${SSH_OPTS_ARRAY[@]}" "${SSH_DIRECT_OPTS[@]}" "$BOX_ADDR" "ls -1 \"$remote_recycle_path\"" 2>/dev/null) || true - if [[ -z "$date_folders" ]]; then - echo "❌ No dated folders found in the recycle bin. Nothing to restore." >&2 - return 1 + local date_folders; date_folders=$(ssh "${SSH_OPTS_ARRAY[@]}" "${SSH_DIRECT_OPTS[@]}" "$BOX_ADDR" "ls -1 \"$remote_recycle_path\"" 2>/dev/null) || true + local valid_folders=() + for f in $date_folders; do + if [[ "$f" =~ ^[0-9]{4}-[0-9]{2}-[0-9]{2}_[0-9]{6}$ ]]; then + valid_folders+=( "$f" ) + fi + done + date_folders=("${valid_folders[@]}") + if [[ ${#date_folders[@]} -eq 0 ]]; then + echo "❌ No validly-named backup folders found in the recycle bin." >&2; return 1 fi printf "${C_YELLOW}Select a backup run (date_time) to browse:${C_RESET}\n" - select date_choice in $date_folders "Cancel"; do + select date_choice in "${date_folders[@]}" "Cancel"; do if [[ "$date_choice" == "Cancel" ]]; then echo "Restore cancelled."; return 0; elif [[ -n "$date_choice" ]]; then break; else echo "Invalid selection. Please try again."; fi @@ -374,61 +379,117 @@ run_restore_mode() { local remote_date_path="${remote_recycle_path}/${date_choice}" printf "${C_BOLD}--- Files available from ${date_choice} (showing first 20) ---${C_RESET}\n" local remote_listing_source="${BOX_ADDR}:${remote_date_path}/" - rsync -r -n --out-format='%n' -e "$SSH_CMD" "$remote_listing_source" . 2>/dev/null | head -n 20 || echo "No files found for this date." + rsync -r -n --out-format='%n' -e "$SSH_CMD" "$remote_listing_source" /dev/null | head -n 20 || echo "No files found for this date." printf "${C_BOLD}--------------------------------------------------------${C_RESET}\n" - printf "${C_YELLOW}Enter the full original path of the item to restore (e.g., home/user/file.txt): ${C_RESET}" - read -r specific_path + printf "${C_YELLOW}Enter the full original path of the item to restore (e.g., home/user/file.txt): ${C_RESET}"; read -r specific_path + if [[ "$specific_path" == /* || "$specific_path" =~ (^|/)\.\.(/|$) ]]; then + echo "❌ Invalid restore path: must be relative and contain no '..'" >&2; return 1 + fi specific_path=$(echo "$specific_path" | sed 's#^/##') if [[ -z "$specific_path" ]]; then echo "❌ Path cannot be empty. Aborting."; return 1; fi full_remote_source="${BOX_ADDR}:${remote_date_path}/${specific_path}" - if ! rsync -r -n -e "$SSH_CMD" "$full_remote_source" . >/dev/null 2>&1; then - echo "❌ ERROR: The path '${specific_path}' was not found in the recycle bin for ${date_choice}. Aborting." >&2 - return 1 + if ! rsync -r -n -e "$SSH_CMD" "$full_remote_source" /dev/null >/dev/null 2>&1; then + echo "❌ ERROR: The path '${specific_path}' was not found in the recycle bin for ${date_choice}. Aborting." >&2; return 1 fi - default_local_dest="/${specific_path}" - item_for_display="(from Recycle Bin) '${specific_path}'" + default_local_dest="/${specific_path}"; item_for_display="(from Recycle Bin) '${specific_path}'" elif [[ "$dir_choice" == "Cancel" ]]; then - echo "Restore cancelled." - return 0 + echo "Restore cancelled."; return 0 else item_for_display="the entire directory '${dir_choice}'" while true; do - printf "\n${C_YELLOW}Restore the entire directory or a specific file/subfolder? [entire/specific]: ${C_RESET}" - read -r choice + printf "\n${C_YELLOW}Restore the entire directory or a specific file/subfolder? [entire/specific]: ${C_RESET}"; read -r choice case "$choice" in - entire) - is_full_directory_restore=true - break - ;; + entire) is_full_directory_restore=true; break ;; specific) - local specific_path_prompt - printf -v specific_path_prompt "Enter the path relative to '%s' to restore: " "$dir_choice" - printf "${C_YELLOW}%s${C_RESET}" "$specific_path_prompt" - read -er specific_path + printf -v specific_path_prompt "Enter the path relative to '%s' to restore: " "$dir_choice"; printf "${C_YELLOW}%s${C_RESET}" "$specific_path_prompt"; read -er specific_path + if [[ "$specific_path" == /* || "$specific_path" =~ (^|/)\.\.(/|$) ]]; then + echo "❌ Invalid restore path: must be relative and contain no '..'" >&2; return 1 + fi specific_path=$(echo "$specific_path" | sed 's#^/##') if [[ -n "$specific_path" ]]; then - restore_path="$specific_path" - item_for_display="'$restore_path' from '${dir_choice}'" - break + restore_path="$specific_path"; item_for_display="'$restore_path' from '${dir_choice}'"; break else echo "Path cannot be empty. Please try again or choose 'entire'." - fi - ;; + fi ;; *) echo "Invalid choice. Please answer 'entire' or 'specific'." ;; esac done local relative_path="${dir_choice#*./}" - full_remote_source="${REMOTE_TARGET}${relative_path}${restore_path}" + local remote_base="${REMOTE_TARGET%/}" + full_remote_source="${remote_base}/${relative_path#/}" + if [[ -n "$restore_path" ]]; then + full_remote_source="${full_remote_source%/}/${restore_path#/}" + fi if [[ -n "$restore_path" ]]; then - default_local_dest=$(echo "${dir_choice}${restore_path}" | sed 's#/\./#/#') + default_local_dest=$(echo "${dir_choice}${restore_path}" | sed 's#/\./#/#g') else - default_local_dest=$(echo "$dir_choice" | sed 's#/\./#/#') + default_local_dest=$(echo "$dir_choice" | sed 's#/\./#/#g') fi fi - local final_dest - printf "\n${C_YELLOW}Enter the destination path.\n${C_DIM}Press [Enter] to use the original location (%s):${C_RESET} " "$default_local_dest" - read -r final_dest + local final_dest + printf "\n%s\n" "${C_BOLD}--------------------------------------------------------" + printf "%s\n" " Restore Destination" + printf "%s\n" "--------------------------------------------------------${C_RESET}" + printf "%s\n\n" "Enter the absolute destination path for the restore." + printf "%s\n" "${C_YELLOW}Default (original location):${C_RESET}" + printf "${C_CYAN}%s${C_RESET}\n\n" "$default_local_dest" + printf "%s\n" "Press [Enter] to use the default path, or enter a new one." + read -rp "> " final_dest : "${final_dest:=$default_local_dest}" + local path_validation_attempts=0 + local max_attempts=5 + while true; do + ((path_validation_attempts++)) + if (( path_validation_attempts > max_attempts )); then + printf "\n${C_RED}❌ Too many invalid attempts. Exiting restore mode.${C_RESET}\n"; return 1 + fi + if [[ "$final_dest" != "/" ]]; then final_dest="${final_dest%/}"; fi + local parent_dir; parent_dir=$(dirname -- "$final_dest") + if [[ "$final_dest" != /* ]]; then + printf "\n${C_RED}❌ Error: Please provide an absolute path (starting with '/').${C_RESET}\n" + elif [[ -e "$final_dest" && ! -d "$final_dest" ]]; then + printf "\n${C_RED}❌ Error: The destination '%s' exists but is a file. Please choose a different path.${C_RESET}\n" "$final_dest" + elif [[ -e "$parent_dir" && ! -w "$parent_dir" ]]; then + printf "\n${C_RED}❌ Error: The parent directory '%s' exists but is not writable.${C_RESET}\n" "$parent_dir" + elif [[ -d "$final_dest" ]]; then + printf "${C_GREEN}✅ Destination '%s' exists and is accessible.${C_RESET}\n" "$final_dest" + if [[ "$final_dest" != "$default_local_dest" && -z "$restore_path" ]]; then + local warning_msg="⚠️ WARNING: Custom destination directory already exists. Files may be overwritten." + printf "${C_YELLOW}%s${C_RESET}\n" "$warning_msg"; log_message "$warning_msg" + fi + break + else + printf "\n${C_YELLOW}⚠️ The destination '%s' does not exist.${C_RESET}\n" "$final_dest" + printf "${C_YELLOW}Choose an action:${C_RESET}\n" + PS3="Your choice: " + select action in "Create the destination path" "Enter a different path" "Cancel"; do + case "$action" in + "Create the destination path") + if mkdir -p "$final_dest"; then + printf "${C_GREEN}✅ Successfully created directory '%s'.${C_RESET}\n" "$final_dest" + if [[ "${is_full_directory_restore:-false}" == "true" ]]; then + chmod 700 "$final_dest"; log_message "Set permissions to 700 on newly created restore directory: $final_dest" + else + chmod 755 "$final_dest" + fi + break 2 + else + printf "\n${C_RED}❌ Failed to create directory '%s'. Check permissions.${C_RESET}\n" "$final_dest"; break + fi ;; + "Enter a different path") break ;; + "Cancel") echo "Restore cancelled by user."; return 0 ;; + *) echo "Invalid option. Please try again." ;; + esac + done + PS3="#? " + fi + if (( path_validation_attempts < max_attempts )); then + printf "\n${C_YELLOW}Please enter a new destination path: ${C_RESET}"; read -r final_dest + if [[ -z "$final_dest" ]]; then + final_dest="$default_local_dest"; printf "${C_DIM}Empty input, using default location: %s${C_RESET}\n" "$final_dest" + fi + fi + done local extra_rsync_opts=() local dest_user="" if [[ "$final_dest" == /home/* ]]; then @@ -436,60 +497,41 @@ run_restore_mode() { if [[ -n "$dest_user" ]] && id -u "$dest_user" &>/dev/null; then printf "${C_CYAN}ℹ️ Home directory detected. Restored files will be owned by '${dest_user}'.${C_RESET}\n" extra_rsync_opts+=("--chown=${dest_user}:${dest_user}") + chown "${dest_user}:${dest_user}" "$final_dest" 2>/dev/null || true else dest_user="" fi fi - local dest_created=false - if [[ ! -e "$final_dest" ]]; then - dest_created=true - fi - local dest_parent - dest_parent=$(dirname "$final_dest") - if ! mkdir -p "$dest_parent"; then - echo "❌ FATAL: Could not create parent destination directory '$dest_parent'. Aborting." >&2 - return 1 - fi - if [[ -n "$dest_user" ]]; then - chown "${dest_user}:${dest_user}" "$dest_parent" - fi - if [[ "$final_dest" != "$default_local_dest" && -d "$final_dest" && -z "$restore_path" ]]; then - local warning_msg="⚠️ WARNING: The custom destination directory '$final_dest' already exists. Files may be overwritten." - echo "$warning_msg"; log_message "$warning_msg" - fi - if [[ "$dest_created" == "true" && "${is_full_directory_restore:-false}" == "true" ]]; then - chmod 700 "$final_dest"; log_message "Set permissions to 700 on newly created restore directory: $final_dest" - fi - printf "Restore destination is set to: ${C_BOLD}%s${C_RESET}\n" "$final_dest" - printf "\n${C_BOLD}${C_YELLOW}--- PERFORMING DRY RUN. NO FILES WILL BE CHANGED. ---${C_RESET}\n" + printf "\n${C_BOLD}Restore Summary:${C_RESET}\n" + printf " Source: %s\n" "$item_for_display" + printf " Destination: ${C_BOLD}%s${C_RESET}\n" "$final_dest" + printf "\n${C_BOLD}${C_YELLOW}--- PERFORMING DRY RUN (NO CHANGES MADE) ---${C_RESET}\n" log_message "Starting restore dry-run of ${item_for_display} from ${full_remote_source} to ${final_dest}" - local rsync_restore_opts=(-avhi --progress --exclude-from="$EXCLUDE_FILE_TMP" -e "$SSH_CMD") + local rsync_restore_opts=(-avhi --safe-links --progress --exclude-from="$EXCLUDE_FILE_TMP" -e "$SSH_CMD") if ! rsync "${rsync_restore_opts[@]}" "${extra_rsync_opts[@]}" --dry-run "$full_remote_source" "$final_dest"; then - echo "❌ DRY RUN FAILED. Rsync reported an error. Aborting." >&2; return 1 + printf "${C_RED}❌ DRY RUN FAILED. Rsync reported an error. Check connectivity and permissions.${C_RESET}\n" >&2 + log_message "Restore dry-run failed for ${item_for_display}"; return 1 fi printf "${C_BOLD}${C_GREEN}--- DRY RUN COMPLETE ---${C_RESET}\n" - local confirmation while true; do - printf "\n${C_YELLOW}Are you sure you want to proceed with restoring %s to '%s'? [yes/no]: ${C_RESET}" "$item_for_display" "$final_dest" - read -r confirmation - - case "$confirmation" in - yes) break ;; - no) echo "Restore aborted by user." ; return 0 ;; - *) echo "Please answer yes or no." ;; + printf "\n${C_YELLOW}Proceed with restoring %s to '%s'? [yes/no]: ${C_RESET}" "$item_for_display" "$final_dest"; read -r confirmation + case "${confirmation,,}" in + yes|y) break ;; + no|n) echo "Restore cancelled by user."; return 0 ;; + *) echo "Please answer 'yes' or 'no'." ;; esac done - printf "\n${C_BOLD}--- PROCEEDING WITH RESTORE... ---${C_RESET}\n" - log_message "Starting REAL restore of ${item_for_display} from ${full_remote_source} to ${final_dest}" + printf "\n${C_BOLD}--- EXECUTING RESTORE ---${C_RESET}\n" + log_message "Starting actual restore of ${item_for_display} from ${full_remote_source} to ${final_dest}" if rsync "${rsync_restore_opts[@]}" "${extra_rsync_opts[@]}" "$full_remote_source" "$final_dest"; then log_message "Restore completed successfully." printf "${C_GREEN}✅ Restore of %s to '%s' completed successfully.${C_RESET}\n" "$item_for_display" "$final_dest" send_notification "Restore SUCCESS: ${HOSTNAME}" "white_check_mark" "${NTFY_PRIORITY_SUCCESS}" "success" "Successfully restored ${item_for_display} to ${final_dest}" else - log_message "Restore FAILED with rsync exit code $?." + local rsync_exit_code=$? + log_message "Restore FAILED with rsync exit code ${rsync_exit_code}." printf "${C_RED}❌ Restore FAILED. Check the rsync output and log for details.${C_RESET}\n" - send_notification "Restore FAILED: ${HOSTNAME}" "x" "${NTFY_PRIORITY_FAILURE}" "failure" "Restore of ${item_for_display} to ${final_dest} failed." - return 1 + send_notification "Restore FAILED: ${HOSTNAME}" "x" "${NTFY_PRIORITY_FAILURE}" "failure" "Restore of ${item_for_display} to ${final_dest} failed (exit code: ${rsync_exit_code})"; return 1 fi } run_recycle_bin_cleanup() { diff --git a/backup_script.sh.sha256 b/backup_script.sh.sha256 index 93fceb0..a9f2bae 100644 --- a/backup_script.sh.sha256 +++ b/backup_script.sh.sha256 @@ -1 +1 @@ -6f989d6cc03d6f507ee48ba93b6b44645ee6e3aaf1da0ee73577056f6c13ea84 backup_script.sh +0860ceaaa6356cc246b3e5eb540aa25f92512087f7087b6c16abee7294e333d6 backup_script.sh