From: Chris Koeritz <fred@gruntose.com>
Date: Thu, 2 Nov 2017 13:10:54 +0000 (-0400)
Subject: improvement in chowning
X-Git-Tag: 2.140.90~116
X-Git-Url: https://feistymeow.org/gitweb/?a=commitdiff_plain;h=895fce4bfda94674dbfd44ad6c9fa8ef13cbe465;p=feisty_meow.git

improvement in chowning

need to make sure anything created as sudo gets fixed to be in the USER's ownership.
---

diff --git a/scripts/system/add_apache_site.sh b/scripts/system/add_apache_site.sh
index 13015569..fe77ae7c 100644
--- a/scripts/system/add_apache_site.sh
+++ b/scripts/system/add_apache_site.sh
@@ -105,15 +105,18 @@ echo full path is $full_path
   fi
   # now give the web server some access to the folder.  this is crucial since the folders
   # can be hosted in any user folder, and the group permissions will usually be only for the user.
-  chown -R $USER:www-data "$full_path"
+  chown -R $USER:www-data "$BASE_PATH"
   check_result "Failed to set www-data as the owner on the path: $full_path"
   # note that web serving will also hose up unless the path to the folder is writable.  so we walk backwards
   # and make sure group access is available.
   local chow_path="$full_path"
-  while [[ $chow_path != $BASE_PATH ]]; do
+  while [[ $chow_path != $HOME ]]; do
 echo chow path is now $chow_path
     chmod -R g+rx "$chow_path"
-    check_result "Failed to add group permissions for www-data on the path: $full_path"
+    check_result "Failed to add group permissions for www-data on the path: $chow_path"
+    # reassert the user's ownership of any directories we might have just created.
+    chown $USER "$chow_path"
+    check_result "changing ownership to user failed on the path: $chow_path"
     chow_path="$(dirname "$chow_path")"
   done
 }