Merge pull request #4354 from jdayamx/Add-log-file-permissions

Added log file permissions

CHANGELOG

@@ -9,6 +9,7 @@ Version 1.1.24 under development
 - Bug #4355: Fix errorhandler missing backtrace entries (georaldc, marcovtwout)
 - Enh #4349: Added CHtml option to omit type attribute from <script> tag (mohamedmalki, marcovtwout)
 - Enh #4351: Added CHtml option to omit CDATA wrapper from <script> and <style> contents (marcovtwout)
+- Enh #4354: Allow to set log file permissions for CFileLogRoute (jdayamx)
 - Chg #4344: Upgraded jQuery to 1.12.4 (marcovtwout)
 - Chg #4344: Upgraded jQuery UI to 1.12.1 (marcovtwout)
 

framework/logging/CFileLogRoute.php

@@ -55,7 +55,12 @@ class CFileLogRoute extends CLogRoute
 	 * @since 1.1.14
 	 */
 	public $rotateByCopy=false;
-
+	/**
+	 * @var integer the permission to be set for newly created log files.
+	 * This value will be used by PHP chmod() function. No umask will be applied.
+	 * If not set, the permission will be determined by the current environment.
+	 */
+	public $chmod;
 	/**
 	 * Initializes the route.
 	 * This method is invoked after the route is created by the route manager.
@@ -163,6 +168,8 @@ protected function processLogs($logs)
 			@flock($fp,LOCK_UN);
 			@fclose($fp);
 		}
+		if($this->chmod !== null)
+			@chmod($logFile, $this->chmod);
 	}
 
 	/**