Skip to content

Error checking #71

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
legioner0 wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Error checking #71

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3e78c9f
1. Non-zero exit status on config/parameters error.
legioner0 Jan 25, 2017
fff7d46
Show help without error
legioner0 Jan 25, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
77 changes: 46 additions & 31 deletions src/zmbkpose
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,6 @@ echo "Usage:

"
# TODO Implements a hidden "Disaster Recovery" option + confirmations
exit 0
}


Expand Down Expand Up @@ -131,7 +130,6 @@ echo "Usos:

"
# TODO: Deixar uma opção de "Disaster Recovery" oculta + confirmações
exit 0
}


Expand Down Expand Up @@ -334,16 +332,16 @@ backup_accounts_parallel ()
incremental_all_accounts ()
{

FULLSESSIONLABEL=$(grep "SESSION: full-" $WORKDIR/sessions.txt | tail -1 | awk '{print $2}')
FULLSESSIONLABEL=$(grep "SESSION: full-" $WORKDIR/sessions.txt 2>/dev/null | tail -1 | awk '{print $2}')

if ! [ -z "$FULLSESSIONLABEL" ]; then
if ! [ -d "$WORKDIR/$FULLSESSIONLABEL" ]; then
echo "Directory $WORKDIR/$FULLSESSIONLABEL doesn't exist. Impossible to continue."
exit 0
exit 1
fi
else
echo "No full backup found. Impossible to continue."
exit 0
exit 1
fi


Expand Down Expand Up @@ -512,7 +510,7 @@ incremental_accounts ()

list_sessions ()
{
egrep 'SESSAO:|SESSION:' $WORKDIR/sessions.txt| egrep 'iniciada|started' | awk '{print $2}' | sort | uniq #egrep for with OR, for previosly compatibility
egrep 'SESSAO:|SESSION:' $WORKDIR/sessions.txt 2>/dev/null | egrep 'iniciada|started' | awk '{print $2}' | sort | uniq #egrep for with OR, for previosly compatibility
exit 0
}

Expand All @@ -531,7 +529,7 @@ restore_accounts ()
break
fi

grep $MAIL $WORKDIR/sessions.txt | grep -v ^WARN: > $BKPSACCOUNT
grep $MAIL $WORKDIR/sessions.txt 2>/dev/null | grep -v ^WARN: > $BKPSACCOUNT

if ! [ -s $BKPSACCOUNT ]; then
echo "$MAIL: No backup found. Impossible to restor"
Expand Down Expand Up @@ -580,10 +578,10 @@ restore_on_box()

if [ -n $DEST ]; then

grep $MAIL $WORKDIR/sessions.txt | grep -e ^inc- -e ^full- > $BKPSACCOUNT
grep $MAIL $WORKDIR/sessions.txt 2>/dev/null | grep -e ^inc- -e ^full- > $BKPSACCOUNT
if ! [ -s $BKPSACCOUNT ]; then
echo "$MAIL: No backup found. Impossible to restore."
exit 1
exit 1
else
echo "Sessions found $(cat $BKPSACCOUNT | awk -F: '{print $1}')"

Expand Down Expand Up @@ -627,7 +625,7 @@ restore_LDAP_account ()
echo "Account $MAIL exists. Use zmbkpose -r $MAIL SESSION_name."
((K = K+1))
else
grep $MAIL $WORKDIR/sessions.txt | grep -e ^inc- -e ^full- > $BKPSACCOUNT
grep $MAIL $WORKDIR/sessions.txt 2>/dev/null | grep -e ^inc- -e ^full- > $BKPSACCOUNT
if ! [ -s $BKPSACCOUNT ]; then
echo "$MAIL: No backup found. Impossible to restore."
((K = K+1))
Expand All @@ -648,7 +646,7 @@ restore_LDAP_account ()
fi
fi
else
SESSIONACCOUNT=$(grep $2 $WORKDIR/sessions.txt | grep $MAIL | tail -1 | awk -F: '{print $1}')
SESSIONACCOUNT=$(grep $2 $WORKDIR/sessions.txt 2>/dev/null | grep $MAIL | tail -1 | awk -F: '{print $1}')
if [ -z $SESSIONACCOUNT ]; then
echo "$MAIL: Session $2 doesn't exist or account not present. Impossible to restore."
((K = K+1))
Expand All @@ -669,7 +667,7 @@ restore_LDAP_account ()

restore_all_accounts_parallel ()
{
for BKP in $(grep -e ^full- -e ^inc- $WORKDIR/sessions.txt); do
for BKP in $(grep -e ^full- -e ^inc- $WORKDIR/sessions.txt 2>/dev/null); do
STAT=$(ps aux | grep -i curl | grep -v grep| wc -l)

if [ $STAT -lt $MAX_PARALLEL_PROCESS ];then
Expand Down Expand Up @@ -721,7 +719,7 @@ restore_all_accounts_parallel ()

restore_alias()
{
for BKP in $(grep -e "^alias-" $WORKDIR/sessions.txt); do
for BKP in $(grep -e "^alias-" $WORKDIR/sessions.txt 2>/dev/null); do
SESSAOCONTA=$(echo $BKP | awk -F: '{print $1}')
MAIL=$(echo $BKP | awk -F: '{print $2}')
$(which ldapadd) -x -H $LDAPMASTERSERVER -D $LDAPZIMBRADN -c -w $LDAPZIMBRAPASS -f $WORKDIR/$SESSAOCONTA/$MAIL.ldiff
Expand All @@ -731,7 +729,7 @@ restore_alias()

restore_distribution_list()
{
for BKP in $(grep -e "^distlist-" $WORKDIR/sessions.txt); do
for BKP in $(grep -e "^distlist-" $WORKDIR/sessions.txt 2>/dev/null); do
SESSAOCONTA=$(echo $BKP | awk -F: '{print $1}')
MAIL=$(echo $BKP | awk -F: '{print $2}')
$(which ldapadd) -x -H $LDAPMASTERSERVER -D $LDAPZIMBRADN -c -w $LDAPZIMBRAPASS -f $WORKDIR/$SESSAOCONTA/$MAIL.ldiff
Expand Down Expand Up @@ -766,12 +764,12 @@ exit 0

rotate_backup ()
{
cp $WORKDIR/sessions.txt $WORKDIR/sessions.txt.old
cp $WORKDIR/sessions.txt $WORKDIR/sessions.txt.old 2>/dev/null

grep SESS $WORKDIR/sessions.txt | awk '{print $2}'| while read line; do
grep SESS $WORKDIR/sessions.txt 2>/dev/null | awk '{print $2}'| while read line; do
if [ "$(echo ${line} | cut -d- -f2)" -lt "$1" ]; then
[ -d $WORKDIR/"${line}" ] && rm -rf $WORKDIR/"${line}" && echo Exclude session "${line}"
grep -v "${line}" $WORKDIR/sessions.txt > $WORKDIR/.sessions.txt
grep -v "${line}" $WORKDIR/sessions.txt 2>/dev/null > $WORKDIR/.sessions.txt
mv $WORKDIR/.sessions.txt $WORKDIR/sessions.txt
fi
done
Expand All @@ -782,56 +780,61 @@ rotate_backup ()

###### MAIN ############
# Here the code loads the config file
source /etc/zmbkpose/zmbkpose.conf
CONF="/etc/zmbkpose/zmbkpose.conf"
if [ ! -f "$CONF" ]; then
echo "No config file $CONF found! You need to run install.sh"
exit 1
fi
source $CONF

if ! [ -z "$BACKUPUSER" ]; then
if [ "$(id -u)" != "$(id -u $BACKUPUSER)" ]; then
echo "You need to be $BACKUPUSER to run this script"
exit 0
exit 1
fi
else
echo "You need to define the variable BACKUPUSER"
exit 0
exit 1
fi

if ! [ -z "$WORKDIR" ]; then
if ! [ -d "$WORKDIR" ]; then
echo "The directory $WORKDIR doesn't exist"
exit 0
exit 1
fi
else
echo "You need to define the variable WORKDIR"
exit 0
exit 1
fi

if [ -z "$ADMINUSER" ]; then
echo "You need to define the variable ADMINUSER"
exit 0
exit 1
fi

if [ -z "$ADMINPASS" ]; then
echo "You need to define the variable ADMINPASS"
exit 0
exit 1
fi

if [ -z "$LDAPMASTERSERVER" ]; then
echo "You need to define the variable LDAPMASTERSERVER"
exit 0
exit 1
fi

if [ -z "$LDAPZIMBRADN" ]; then
echo "You need to define the variable LDAPZIMBRADN"
exit 0
exit 1
fi

if [ -z "$LDAPZIMBRAPASS" ]; then
echo "You need to define the variable LDAPZIMBRAPASS"
exit 0
exit 1
fi

if [ -z "$LOGFILE" ]; then
echo "You need to define the variable LOGFILE"
exit 0
exit 1
fi

# Criticize the parameters passed on the command line
Expand All @@ -851,6 +854,7 @@ case "$1" in
else
echo "Incorrect $@ params. Read --help."
show_help
exit 1
fi
fi

Expand All @@ -864,6 +868,7 @@ case "$1" in
fi
echo "Incorrect $@ params. Read the --help."
show_help
exit 1
fi
;;
"-l" )
Expand All @@ -872,25 +877,29 @@ case "$1" in
else
echo "Incorrect $@ params. Read the --help."
show_help
exit 1
fi
;;
"-r" )
if [ -z "$2" ]; then
echo "Incorrect $@ params. Read the --help."
show_help
exit 1
else
if [ -z "$4" ]; then
restore_accounts $2 $3
else
echo "Incorrect $@ params. Read the --help."
show_help
exit 1
fi
fi
;;
"-restoreAccount" )
if [ -z "$2" ]; then
echo "Incorrect $@ params. Read the --help."
show_help
exit 1
else
if [ -z "$3" ]; then
restore_LDAP_account $2
Expand All @@ -900,6 +909,7 @@ case "$1" in
else
echo "Incorrect $@ params. Read the --help."
show_help
exit 1
fi
fi
fi
Expand Down Expand Up @@ -929,7 +939,7 @@ case "$1" in
if [ -z "$2" ]; then
echo "Parameter -d accepts only number of days, or a number followed by weeks or months"
show_help
exit 0
exit 1
fi
if [ $2 -eq $2 2> /dev/null ]; then
if [ -z "$3" ]; then
Expand All @@ -945,21 +955,26 @@ case "$1" in
* )
echo "Parameter -d accepts only number of days, or a number followed by weeks or months"
show_help
exit 0
exit 1
;;
esac
fi
rotate_backup $OLDEST
else
echo "Parameter -d accepts only number of days, or a number followed by weeks or months"
show_help
exit 0
exit 1
fi
;;

"-h" )
show_help
;;

* )
echo "Incorrect parameters $@. See help."
show_help
exit 1
;;
esac
exit 0
Expand Down