fix(logging): Fix array to string conversion in errorlog writer

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

Author: ChristophWurst

lib/private/Log/Errorlog.php

@@ -28,24 +28,27 @@
 
 namespace OC\Log;
 
+use OC\SystemConfig;
 use OCP\Log\IWriter;
 
-class Errorlog implements IWriter {
+class Errorlog extends LogDetails implements IWriter {
 
 	/** @var string */
 	protected $tag;
 
-	public function __construct(string $tag = 'nextcloud') {
+	public function __construct(SystemConfig $config, string $tag = 'nextcloud') {
+		parent::__construct($config);
 		$this->tag = $tag;
 	}
 
 	/**
-	 * write a message in the log
+	 * Write a message in the log
+	 *
 	 * @param string $app
-	 * @param string $message
+	 * @param string|array $message
 	 * @param int $level
 	 */
 	public function write(string $app, $message, int $level) {
-		error_log('[' . $this->tag . ']['.$app.']['.$level.'] '.$message);
+		error_log('[' . $this->tag . ']['.$app.']['.$level.'] '.$this->logDetailsAsJSON($app, $message, $level));
 	}
 }

lib/private/Log/LogFactory.php

@@ -49,7 +49,7 @@ public function __construct(IServerContainer $c, SystemConfig $systemConfig) {
 	public function get(string $type):IWriter {
 		switch (strtolower($type)) {
 			case 'errorlog':
-				return new Errorlog();
+				return new Errorlog($this->systemConfig);
 			case 'syslog':
 				return $this->c->resolve(Syslog::class);
 			case 'systemd':
@@ -73,7 +73,7 @@ public function getCustomLogger(string $path):ILogger {
 	protected function createNewLogger(string $type, string $tag, string $path): IWriter {
 		switch (strtolower($type)) {
 			case 'errorlog':
-				return new Errorlog($tag);
+				return new Errorlog($this->systemConfig, $tag);
 			case 'syslog':
 				return new Syslog($this->systemConfig, $tag);
 			case 'systemd':

lib/private/Log/Syslog.php

@@ -53,7 +53,7 @@ public function __destruct() {
 	/**
 	 * write a message in the log
 	 * @param string $app
-	 * @param string $message
+	 * @param string|array $message
 	 * @param int $level
 	 */
 	public function write(string $app, $message, int $level) {

lib/private/Log/Systemdlog.php

@@ -72,7 +72,7 @@ public function __construct(SystemConfig $config, ?string $tag = null) {
 	/**
 	 * Write a message to the log.
 	 * @param string $app
-	 * @param string $message
+	 * @param string|array $message
 	 * @param int $level
 	 */
 	public function write(string $app, $message, int $level) {

lib/public/Log/IWriter.php

@@ -30,6 +30,10 @@
 interface IWriter {
 	/**
 	 * @since 14.0.0
+	 *
+	 * @param string $app
+	 * @param string|array $message
+	 * @param int $level
 	 */
 	public function write(string $app, $message, int $level);
 }