Merge branch 'buster_web-rotate-logs' into 'buster-testing'

Fix web-rotate-logs (buster)

See merge request sympl/sympl!99

web/debian/changelog

+sympl-web (10.0.190709.0) stable; urgency=medium
+
+  * Updated sympl-web-rotate-logs to support new ownership
+  * Reload Apache when rotating logs, rather than the loggers.
+
+ -- Paul Cammish <sympl@kelduum.net>  Tue, 09 Jul 2019 19:18:01 +0100
+
 sympl-web (10.0.190704.0) stable; urgency=medium
 
   * Rewrote Apache configs

web/sbin/sympl-web-rotate-logs

@@ -227,14 +227,6 @@ Symbiosis::Domains.each(prefix) do |domain|
   #
   rotated = true
 
-  #
-  # Drop privileges
-  #
-  if 0 == Process.uid
-    Process::Sys.setegid(domain.gid)
-    Process::Sys.seteuid(domain.uid)
-  end
-
   #
   #  Process these files backward so we don't over-write anything.
   #
@@ -273,13 +265,6 @@ Symbiosis::Domains.each(prefix) do |domain|
     end
   end
   
-  #
-  # Restore back to root.
-  #
-  if 0 == Process.uid
-    Process::Sys.seteuid(0)
-    Process::Sys.setegid(0)
-  end
 end
 
 #
@@ -290,8 +275,8 @@ if ( rotated )
   # TODO:  Really we want to send HUP to those processes which have the
   # logfiles we've rotated open, rather than blindly stabbing things.
   #
-  verbose( "Since we rotated we now restart the httpd logger")
-  Kernel.system( "killall -HUP sympl-web-logger" )
+  verbose( "Since we rotated we now reload Apache")
+  Kernel.system( "/usr/sbin/service apache2 reload" )
   
   #
   # Add similar arguments on to the generate-stats command as wot we received.