From c8fa8ed1b443ea8b3a9f97c14a4031e50e02f584 Mon Sep 17 00:00:00 2001 From: Beau Raines Date: Thu, 28 Mar 2019 15:14:36 -0700 Subject: [PATCH] Switches to MySQL Options files for root credentials Removes the root user/pass from the configuration file and uses a mysqloptions file https://dev.mysql.com/doc/refman/8.0/en/option-files.html This prevents warnings from mysql about using a pssword on the command line Sample File make sure its not publicly readable, like your SSH keys. ``` [client] user=root password=your_password ``` Closes #14 --- backup.conf | 3 +-- backupIncremental.sh | 4 ++-- backupWeekly.sh | 4 ++-- 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/backup.conf b/backup.conf index 195d80a..dd4b9cd 100644 --- a/backup.conf +++ b/backup.conf @@ -12,8 +12,7 @@ MinecraftInstalled=0 #MySQL Configuration backupMySQL=1 -MYSQLROOTUSER= -MYSQLROOTPASS= +MYSQLOPTIONSFILE= # PATH TO mysql options file with root credentials #SVN Configuration backupSVN=1 diff --git a/backupIncremental.sh b/backupIncremental.sh index 554c10f..513924d 100755 --- a/backupIncremental.sh +++ b/backupIncremental.sh @@ -83,11 +83,11 @@ if [ $backupMySQL -eq 1 ] then echo -n "Backing up mysql databases..." echo "Backing up mysql databases..." >> $BACKUP_DIR/log/backup_$DATES.log -for i in `mysql -u$MYSQLROOTUSER -p$MYSQLROOTPASS -BNe 'select schema_name from information_schema.schemata;'` +for i in `mysql --defaults-file=$MYSQLOPTIONSFILE -BNe 'select schema_name from information_schema.schemata;'` do BACKUPMODULE="mysql database schema $i" echo "Backing up $BACKUPMODULE" - mysqldump -R --add-drop-table -v --opt --lock-all-tables --log-error=$BACKUP_DIR/log/backup_$DATES.log -u $MYSQLROOTUSER -p$MYSQLROOTPASS $i | gzip > $BACKUP_DIR/mysql/$i.dmp.sql.gz + mysqldump --defaults-file=$MYSQLOPTIONSFILE -R --add-drop-table -v --opt --lock-all-tables --log-error=$BACKUP_DIR/log/backup_$DATES.log $i | gzip > $BACKUP_DIR/mysql/$i.dmp.sql.gz rcCheck $? sleep 10 done diff --git a/backupWeekly.sh b/backupWeekly.sh index ee80319..c86b9c5 100755 --- a/backupWeekly.sh +++ b/backupWeekly.sh @@ -100,11 +100,11 @@ if [ $backupMySQL -eq 1 ] then echo -n "Backing up mysql databases..." echo "Backing up mysql databases..." >> $BACKUP_DIR/log/backup_$DATES.log -for i in `mysql -u$MYSQLROOTUSER -p$MYSQLROOTPASS -BNe 'select schema_name from information_schema.schemata;'` +for i in `mysql --defaults-file=$MYSQLOPTIONSFILE -BNe 'select schema_name from information_schema.schemata;'` do BACKUPMODULE="mysql database schema $i" echo "Backing up $BACKUPMODULE" - mysqldump -R --add-drop-table -v --opt --lock-all-tables --log-error=$BACKUP_DIR/log/backup_$DATES.log -u $MYSQLROOTUSER -p$MYSQLROOTPASS $i | gzip > $BACKUP_DIR/mysql/$i.dmp.sql.gz + mysqldump --defaults-file=$MYSQLOPTIONSFILE -R --add-drop-table -v --opt --lock-all-tables --log-error=$BACKUP_DIR/log/backup_$DATES.log $i | gzip > $BACKUP_DIR/mysql/$i.dmp.sql.gz rcCheck $? sleep 10 done