From 82f69d7216672cd010c1ee6824feed70b6196b85 Mon Sep 17 00:00:00 2001 From: Mark Friedrich Date: Sun, 21 Jul 2019 15:50:28 +0200 Subject: [PATCH] - fixed `chmod(0666)` for log log files --- app/Component/Handler/LogFileHandler.php | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/app/Component/Handler/LogFileHandler.php b/app/Component/Handler/LogFileHandler.php index 22a759b..a47ff7b 100644 --- a/app/Component/Handler/LogFileHandler.php +++ b/app/Component/Handler/LogFileHandler.php @@ -44,11 +44,15 @@ class LogFileHandler { public function write(array $log){ $log = (string)json_encode($log, JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE); if( !empty($log) ){ - $stream = fopen($this->stream, 'a'); - flock($stream, LOCK_EX); - fwrite($stream, $log . PHP_EOL); - flock($stream, LOCK_UN); - fclose($stream); + if($stream = fopen($this->stream, 'a')){ + flock($stream, LOCK_EX); + fwrite($stream, $log . PHP_EOL); + flock($stream, LOCK_UN); + fclose($stream); + + // logs should be writable for non webSocket user too + @chmod($this->stream, 0666); + } } }