¿Mi script de backup en Bash es seguro? Necesito revisión de lógica

Designer

He escrito un script para hacer backups incrementales con rsync y comprimirlos. Funciona, pero me preocupa que falle en casos extremos (permisos, rutas con espacios, etc.). Aquí el código:

#!/bin/bash
BACKUP_DIR='/var/backups'
SOURCE='/home/user/data'
rsync -avz --delete $SOURCE $BACKUP_DIR/$(date +%Y%m%d)
tar -czf $BACKUP_DIR/backup-$(date +%Y%m%d).tar.gz $BACKUP_DIR/$(date +%Y%m%d)

¿Qué opinan? ¿Alguna mejora de seguridad o robustez?

Piano

Tu script tiene varios problemas. Primero, no escapas las variables, así que si $SOURCE o $BACKUP_DIR contienen espacios, fallará. Usa comillas dobles: "$SOURCE". Segundo, rsync --delete es peligroso si la fuente no está montada; podrías borrar todo el backup. Añade una comprobación de que la fuente existe.

Además, generas el timestamp dos veces, lo que puede dar inconsistencias si la medianoche cruza entre comandos. Mejor así:

#!/bin/bash
BACKUP_DIR='/var/backups'
SOURCE='/home/user/data'
TIMESTAMP=$(date +%Y%m%d)
TARGET="$BACKUP_DIR/$TIMESTAMP"
if [ ! -d "$SOURCE" ]; then
    echo "Error: Source does not exist" >&2
    exit 1
fi
rsync -avz --delete "$SOURCE/" "$TARGET/"
tar -czf "$BACKUP_DIR/backup-$TIMESTAMP.tar.gz" "$TARGET"

Designer

Gracias por la revisión. Tienes razón con las comillas y el timestamp. Lo de comprobar la fuente antes de rsync es clave. ¿Alguna sugerencia para rotar backups viejos automáticamente?

BoardGames

Sobre rotación, podrías usar find con -mtime. Por ejemplo, para borrar backups de más de 7 días:

find "$BACKUP_DIR" -name "backup-*.tar.gz" -mtime +7 -delete

Pero cuidado: si el backup falla un día, podrías borrar el único que tienes. Mejor mover a un directorio de 'trash' o comprobar que exista al menos un backup reciente antes de borrar.

Surfing

Discrepo con el uso de --delete en rsync incluso con comprobación. Si alguien mueve la fuente temporalmente, podrías perder datos. Recomendaría usar --backup --backup-dir para mantener versiones, o simplemente no usar --delete y hacer backups completos periódicos.

Además, comprimir después de rsync duplica el espacio hasta que termina el tar. Podrías comprimir sobre la marcha con rsync -z o directamente enviar a un archivo comprimido con rsync --out-format y pipe a gzip, pero eso es más complejo.

Designer

Buen punto sobre el espacio duplicado. Creo que para mi caso, la comprobación de fuente es suficiente. Probaré sin --delete y usando find para rotar. ¡Gracias a todos!