From 5d76a55c4666cea7758ea076085bb92784304408 Mon Sep 17 00:00:00 2001
From: Alejandro Celaya <alejandro@alejandrocelaya.com>
Date: Sat, 30 Nov 2019 09:28:09 +0100
Subject: [PATCH] Updated how monolog handlers are registered so that it is
 possible to overwrite them via local config

---
 config/autoload/logger.global.php        | 5 ++++-
 config/autoload/logger.local.php.dist    | 8 ++++----
 docker/config/shlink_in_docker.local.php | 7 +++----
 3 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/config/autoload/logger.global.php b/config/autoload/logger.global.php
index 6b0df063..e612d2f4 100644
--- a/config/autoload/logger.global.php
+++ b/config/autoload/logger.global.php
@@ -48,7 +48,10 @@ return [
 
         'loggers' => [
             'Shlink' => [
-                'handlers' => ['shlink_rotating_handler'],
+                'handlers' => [
+                    // Using a key allows for this to be overwritten
+                    'shlink_handler' => 'shlink_rotating_handler',
+                ],
                 'processors' => ['exception_with_new_line', 'psr3'],
             ],
             'Access' => [
diff --git a/config/autoload/logger.local.php.dist b/config/autoload/logger.local.php.dist
index cf7e4801..403a26fa 100644
--- a/config/autoload/logger.local.php.dist
+++ b/config/autoload/logger.local.php.dist
@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 use Monolog\Handler\StreamHandler;
@@ -9,9 +10,6 @@ $isSwoole = extension_loaded('swoole');
 // For swoole, send logs to standard output
 $logger = $isSwoole ? [
     'handlers' => [
-        'shlink_rotating_handler' => [
-            'level' => Logger::EMERGENCY, // This basically disables regular file logs
-        ],
         'shlink_stdout_handler' => [
             'class' => StreamHandler::class,
             'level' => Logger::DEBUG,
@@ -22,7 +20,9 @@ $logger = $isSwoole ? [
 
     'loggers' => [
         'Shlink' => [
-            'handlers' => ['shlink_stdout_handler'],
+            'handlers' => [
+                'shlink_handler' => 'shlink_stdout_handler',
+            ],
         ],
     ],
 ] : [
diff --git a/docker/config/shlink_in_docker.local.php b/docker/config/shlink_in_docker.local.php
index bd2d0411..176e45f5 100644
--- a/docker/config/shlink_in_docker.local.php
+++ b/docker/config/shlink_in_docker.local.php
@@ -131,9 +131,6 @@ return [
 
     'logger' => [
         'handlers' => [
-            'shlink_rotating_handler' => [
-                'level' => Logger::EMERGENCY, // This basically disables regular file logs
-            ],
             'shlink_stdout_handler' => [
                 'class' => StreamHandler::class,
                 'level' => Logger::INFO,
@@ -144,7 +141,9 @@ return [
 
         'loggers' => [
             'Shlink' => [
-                'handlers' => ['shlink_stdout_handler'],
+                'handlers' => [
+                    'shlink_handler' => 'shlink_stdout_handler',
+                ],
             ],
         ],
     ],