From 4cd48d2ddc3daccaa1ad117002cee49b350f9f26 Mon Sep 17 00:00:00 2001 From: Marian Sigler Date: Wed, 26 Nov 2014 01:24:03 +0100 Subject: make neueliste.bash run as mailman not root; small safety improvements --- neueliste.bash | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/neueliste.bash b/neueliste.bash index b74a606..8299ebc 100755 --- a/neueliste.bash +++ b/neueliste.bash @@ -1,4 +1,10 @@ #!/bin/zsh + +if [ $# -ne 3 ]; then + echo "Interner Fehler: Ungültige Anzahl Parameter" + echo "internal error: invalid number of parameters" +fi + LOCKFILE=/tmp/newlist lockfile-create --retry 1 $LOCKFILE >/dev/null 2>&1 @@ -11,7 +17,7 @@ fi TMPFILE=/tmp/newlistoutput$(date -Is) -if su mailman -c "echo | ~mailman/bin/newlist $1 $2 $3" > $TMPFILE 2>&1 +if echo | ~mailman/bin/newlist -- "$1" "$2" "$3" > $TMPFILE 2>&1 then echo "Erstellung erfolgreich -- creation completed" @@ -25,10 +31,8 @@ else echo "Bitte im Browser auf Zurück klicken und einen anderen Namen für die Mailingliste wählen! Danke.
" echo "Please click on back in your browser and enter another name for your mailinglist. Thank you.
debug info:" else - echo "Oh, das ist was schief gegangen. Bitte mailman@lists.spline.de kontaktieren!
" - echo "Oops, something went wrong here. Please contact the admins at mailman@lists.spline.de !

"
-        cat $TMPFILE
-	echo "
" + echo "Oh, da ist was schief gegangen. Bitte mailman@lists.spline.de kontaktieren!
" + echo "Oops, something went wrong here. Please contact the admins at mailman@lists.spline.de !" cat $TMPFILE | mailx -s "$0: Mailinglistenerstellung fehlgeschlagen 1=($1) 2=($2) 3=($3)" mailman@lists.spline.de fi -- cgit v1.2.3-1-g7c22