Add stdout and stderr log target support to Logger

- Introduced support for `log.output` configuration to specify logging target as `file`, `stdout`, or `stderr`.
- Updated `Logger` to handle new log output targets.
- Added unit test for `stdout` output target.

Author: c15k0

src/base/Logger.php

@@ -19,7 +19,6 @@
 
 
 if (!defined('LOG_DIR')) {
-    GeneratorHelper::createDir(BASE_DIR . DIRECTORY_SEPARATOR . 'logs');
     define('LOG_DIR', BASE_DIR . DIRECTORY_SEPARATOR . 'logs');
 }
 
@@ -29,6 +28,10 @@
 class Logger
 {
     const DEFAULT_NAMESPACE = 'PSFS';
+    private const OUTPUT_FILE = 'file';
+    private const OUTPUT_STDOUT = 'stdout';
+    private const OUTPUT_STDERR = 'stderr';
+
     use SingletonTrait;
 
     /**
@@ -47,6 +50,7 @@ class Logger
      * @var string
      */
     protected string $logLevel;
+    private bool $closeStreamOnDestruct = true;
 
     /**
      * @throws GeneratorException
@@ -56,8 +60,9 @@ class Logger
     public function __construct()
     {
         $args = func_get_args();
-        list($logger, $debug, $path) = $this->setup($args);
-        $this->stream = fopen($path . DIRECTORY_SEPARATOR . date('Ymd') . '.log', 'ab+');
+        list($logger, $debug, $target, $closeStreamOnDestruct) = $this->setup($args);
+        $this->closeStreamOnDestruct = $closeStreamOnDestruct;
+        $this->stream = fopen($target, 'ab+');
         if (false !== $this->stream && is_resource($this->stream)) {
             $this->addPushLogger($logger, $debug);
         } else {
@@ -68,7 +73,9 @@ public function __construct()
 
     public function __destruct()
     {
-        fclose($this->stream);
+        if ($this->closeStreamOnDestruct && is_resource($this->stream)) {
+            fclose($this->stream);
+        }
     }
 
     /**
@@ -108,8 +115,26 @@ private function setup(array $args = [])
             $namespace = $args[0][0] ?? 'PSFS';
             $debug = $args[0][1] ?? true;
         }
+        [$target, $closeStreamOnDestruct] = $this->resolveLogOutputTarget();
+        return [LogHelper::cleanLoggerName($namespace), $debug, $target, $closeStreamOnDestruct];
+    }
+
+    /**
+     * @return array{0:string,1:bool}
+     * @throws GeneratorException
+     */
+    private function resolveLogOutputTarget(): array
+    {
+        $output = strtolower(trim((string)Config::getParam('log.output', self::OUTPUT_FILE)));
+        if ($output === self::OUTPUT_STDOUT) {
+            return ['php://stdout', false];
+        }
+        if ($output === self::OUTPUT_STDERR) {
+            return ['php://stderr', false];
+        }
+
         $path = $this->createLoggerPath();
-        return [LogHelper::cleanLoggerName($namespace), $debug, $path];
+        return [$path . DIRECTORY_SEPARATOR . date('Ymd') . '.log', true];
     }
 
     /**

src/base/config/Config.php

@@ -70,6 +70,7 @@ class Config
         'api.secret', // Secret passphrase to securize the api
         'api.admin', // Enable the autogenerated api admin(wok)
         'log.level', // Max log level(default INFO)
+        'log.output', // Logger sink target (file|stdout|stderr)
         'admin_action', // Default admin url when access to /admin
         'cache.var', // Static cache var
         'twig.autoreload', // Enable or disable auto reload templates for twig

src/base/extension/traits/CssTrait.php

@@ -110,7 +110,7 @@ protected function loopCssLines($file)
     }
 
     /**
-     * @param string|array $source
+     * @param array $source
      * @param string $file
      */
     protected function extractCssResources($source, $file)

tests/base/LoggerTest.php

@@ -90,4 +90,31 @@ public function testLogSetup()
         Config::save($defaultConfig, []);
     }
 
+    public function testLoggerSupportsStdoutOutputTarget(): void
+    {
+        $config = Config::getInstance();
+        $configProperty = new \ReflectionProperty(Config::class, 'config');
+        $configProperty->setAccessible(true);
+        $defaultConfig = $configProperty->getValue($config);
+        $overrideConfig = $defaultConfig;
+        $overrideConfig['log.output'] = 'stdout';
+        $configProperty->setValue($config, $overrideConfig);
+
+        try {
+            $reflection = new \ReflectionClass(Logger::class);
+            /** @var Logger $logger */
+            $logger = $reflection->newInstanceWithoutConstructor();
+
+            $method = new \ReflectionMethod(Logger::class, 'resolveLogOutputTarget');
+            $method->setAccessible(true);
+            [$target, $closeStreamOnDestruct] = $method->invoke($logger);
+
+            $this->assertSame('php://stdout', $target);
+            $this->assertFalse($closeStreamOnDestruct);
+        } finally {
+            Logger::dropInstance();
+            $configProperty->setValue($config, $defaultConfig);
+        }
+    }
+
 }