cclohmar/NextExpense
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

fix: database ownership preserved during backup — app user must be able to write

Browse source 
- Backup now uses chown --reference to preserve original owner
- Found: rsync changed database ownership to root, blocking writes
This commit is contained in:
Claus Lohmar 2026-06-03 07:52:23 +00:00
parent bb34528bf5
commit cb42efa395
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
install.sh
View file

@ -79,6 +79,8 @@ if [ "${1:-}" = "-update" ]; then
sudo_if mkdir -p "$BACKUP_DIR" sudo_if mkdir -p "$BACKUP_DIR"
BACKUP_FILE="$BACKUP_DIR/expenses-$(date +%Y%m%d-%H%M%S).db" BACKUP_FILE="$BACKUP_DIR/expenses-$(date +%Y%m%d-%H%M%S).db"
sudo_if cp "$INSTALL_DIR/expenses.db" "$BACKUP_FILE" sudo_if cp "$INSTALL_DIR/expenses.db" "$BACKUP_FILE"
# Restore ownership so the app user can still write to the database.
sudo_if chown --reference="$INSTALL_DIR/expenses.db" "$BACKUP_FILE" 2>/dev/null || true
info "Database backed up to $BACKUP_FILE" info "Database backed up to $BACKUP_FILE"
fi fi