Permissions / ownership tweaks.

psitem · Jun 30, 2017 · 541575d · 541575d
1 parent 9ed27e3
commit 541575d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/config/scripts/post-config.d/hooks.sh b/config/scripts/post-config.d/hooks.sh
@@ -1,9 +1,12 @@
 #!/bin/bash
 source /config/user-data/edgerouter-backup.conf
 
-# Fix ownership
-sudo chown -R root:vyattacfg /config/userdata
+# Fix ownership / permissions
+sudo chown -R root:vyattacfg /config/user-data
 sudo chown -R root:vyattacfg /config/scripts
+sudo chmod -R ug+w /config/user-data
+sudo chmod -R ug+w /config/scripts
+sudo chmod g-w $SSH_KEYFILE
 
 # Ensure scripts are executable
 sudo chmod +x /config/user-data/hooks/*

diff --git a/config/user-data/hooks/03-edgerouter-backup.sh b/config/user-data/hooks/03-edgerouter-backup.sh
@@ -26,9 +26,9 @@ GIT_COMMIT_MSG="$COMMIT_CMT | by $USER | via $COMMIT_VIA | $TIME"
 
 # Remove temporary files
 #echo "edgerouter-backup: Removing temporary files"
-sudo rm /tmp/edgerouter-backup-$FNAME_CONFIG
-sudo rm /tmp/edgerouter-backup-$FNAME_CLI
-sudo rm /tmp/edgerouter-backup-$FNAME_BACKUP
+sudo rm /tmp/edgerouter-backup-$FNAME_CONFIG &> /dev/null
+sudo rm /tmp/edgerouter-backup-$FNAME_CLI &> /dev/null
+sudo rm /tmp/edgerouter-backup-$FNAME_BACKUP &> /dev/null
 
 
 # Generate temporary config files