From: clinton_admin <clinton_admin@navajos.hcoop.net>
Date: Thu, 20 Dec 2012 08:25:58 +0000 (-0500)
Subject: Correct order of sudo in apache sync logs
X-Git-Url: https://git.hcoop.net/hcoop/scripts.git/commitdiff_plain/2a789dae2af8378cc8c8cbf2bccaa9ed050f440c

Correct order of sudo in apache sync logs
* Modern sudo clears the environment, so we have to run k5start within sudo not outside
---

diff --git a/apache-sync-logs b/apache-sync-logs
index 95b66ef..7ad7668 100755
--- a/apache-sync-logs
+++ b/apache-sync-logs
@@ -75,8 +75,8 @@ for A in $(find $LOCAL_LOG_DIR -mindepth 3 -maxdepth 3 -print); do
         cp -r $LOG_SRC $TMP_DEST
         chown -R $USER:nogroup $TMP_DEST
         chmod -R u=rwX,go=X $TMP_DEST
-        k5start -qtU -f $KEYTAB_DIR/$USER \
-            -- sudo -u $USER rsync -a $TMP_DEST/ $LOG_DEST/
+        sudo -u $USER k5start -qtU -f $KEYTAB_DIR/$USER \
+            -- rsync -a $TMP_DEST/ $LOG_DEST/
         rm -fr $TMP_DEST
         test "$VERBOSE" = "true" && echo "  done."
     fi