Skip to content

Commit 68f954c

Browse files
committed
Fix SQLite3 Backups and clean up temp directory properly
1 parent 0ab0a6d commit 68f954c

File tree

1 file changed

+26
-4
lines changed

1 file changed

+26
-4
lines changed

install/assets/functions/10-db-backup

Lines changed: 26 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -839,16 +839,21 @@ backup_sqlite3() {
839839
db="${db%.*}"
840840
backup_job_filename=sqlite3_${db}_${now}.sqlite3
841841
backup_job_filename_base=sqlite3_${db}.sqlite3
842-
compression
843842
pre_dbbackup "${db}"
844843
write_log notice "Dumping sqlite3 database: '${backup_job_db_host}' ${compression_string}"
845844
if var_true "${DEBUG_BACKUP_SQLITE3}" ; then debug on; fi
846-
silent run_as_user ${play_fair} sqlite3 "${backup_job_db_host}" ".backup ${TEMP_PATH}/backup.sqlite3"
845+
silent ${play_fair} sqlite3 "${backup_job_db_host}" ".backup '${TEMP_PATH}/backup_${now}.sqlite3'"
847846
exit_code=$?
848847
check_exit_code backup "${backup_job_filename}"
849-
run_as_user ${play_fair} cat "${TEMP_PATH}"/backup.sqlite3 | ${dir_compress_cmd} | run_as_user tee "${TEMP_PATH}/${backup_job_filename}" > /dev/null
850-
timer backup finish
848+
if [ ! -f "${TEMP_PATH}"/backup_${now}.sqlite3 ] ; then
849+
print_error "SQLite3 backup failed! Exitting"
850+
return 1
851+
fi
852+
compression
853+
run_as_user ${play_fair} cat "${TEMP_PATH}"/backup_${now}.sqlite3 | ${dir_compress_cmd} | run_as_user tee "${TEMP_PATH}/${backup_job_filename}" > /dev/null
854+
rm -rf "${TEMP_PATH}"/backup_${now}.sqlite3
851855
if var_true "${DEBUG_BACKUP_SQLITE3}" ; then debug off; fi
856+
timer backup finish
852857
file_encryption
853858
generate_checksum
854859
move_dbbackup
@@ -1190,17 +1195,34 @@ db_backup_container_init() {
11901195
debug() {
11911196
case "${1}" in
11921197
off)
1198+
backup_job_log_level=$_original_job_log_log_level}
1199+
CONTAINER_LOG_LEVEL=${_original_container_log_level}
11931200
DEBUG_MODE=${_original_debug_mode}
1201+
SHOW_OUTPUT=${_original_show_output}
1202+
11941203
if var_true "${DEBUG_MODE}" ; then
11951204
set -x
11961205
else
11971206
set +x
11981207
fi
11991208
;;
12001209
on)
1210+
if [ -z "${_original_container_log_level}" ]; then
1211+
_original_container_log_level="${CONTAINER_LOG_LEVEL}"
1212+
fi
1213+
if [ -z "${_original_job_log_level}" ]; then
1214+
_original_job_log_level="${backup_job_log_level}"
1215+
fi
1216+
12011217
if [ -z "${_original_debug_mode}" ]; then
12021218
_original_debug_mode="${DEBUG_MODE}"
12031219
fi
1220+
if [ -z "${_original_show_output}" ]; then
1221+
_original_show_output="${SHOW_OUTPUT}"
1222+
fi
1223+
backup_job_log_level=DEBUG
1224+
CONTAINER_LOG_LEVEL=DEBUG
1225+
SHOW_OUTPUT=TRUE
12041226
set -x
12051227
;;
12061228
esac

0 commit comments

Comments
 (0)