Make error logging configurable

Classes/RedisBackend.php

@@ -53,6 +53,7 @@ class RedisBackend extends IndependentAbstractBackend implements TaggableBackend
     protected ?Iterator\Keyspace $entryKeyspaceIterator = null;
     protected int $entryKeyspaceIteratorKeyPrefixLength = 0;
     protected bool $deduplicateErrors = true;
+    protected bool $logErrors = true;
 
     protected static array $loggedErrors = [];
 
@@ -65,7 +66,7 @@ public function __construct(EnvironmentConfiguration $environmentConfiguration,
         parent::__construct($environmentConfiguration, $options);
         $this->client = $this->getRedisClient();
 
-        if (class_exists('Neos\Flow\Core\Bootstrap') && Bootstrap::$staticObjectManager instanceof ObjectManagerInterface) {
+        if ($this->logErrors && class_exists(Bootstrap::class) && Bootstrap::$staticObjectManager instanceof ObjectManagerInterface) {
             $this->logger = Bootstrap::$staticObjectManager->get(LoggerInterface::class);
             $this->throwableStorage = Bootstrap::$staticObjectManager->get(ThrowableStorageInterface::class);
         }
@@ -497,6 +498,14 @@ public function setDeduplicateErrors($deduplicateErrors): void
         $this->deduplicateErrors = (bool)$deduplicateErrors;
     }
 
+    /**
+     * @param bool $logErrors
+     */
+    public function setLogErrors(bool $logErrors): void
+    {
+        $this->logErrors = $logErrors;
+    }
+
     /**
      * @param string|bool $value
      * @return string|bool
@@ -583,7 +592,9 @@ private function handleThrowable(\Throwable $throwable): void
     {
         $messageHash = md5($throwable->getMessage());
         if (!$this->deduplicateErrors || !array_key_exists($messageHash, static::$loggedErrors)) {
-            $this->logger && $this->logger->error($this->throwableStorage->logThrowable($throwable), LogEnvironment::fromMethodName(__METHOD__));
+            if ($this->logErrors && $this->logger && $this->throwableStorage) {
+                $this->logger && $this->logger->error($this->throwableStorage->logThrowable($throwable), LogEnvironment::fromMethodName(__METHOD__));
+            }
             static::$loggedErrors[$messageHash] = true;
         }
         throw $throwable;

README.md

@@ -93,6 +93,18 @@ Flow_Mvc_Routing_Route:
         deduplicateErrors: false
 ```
 
+If you don't want errors being logged – for example, because you log errors via
+the MultiBackend – you can turn off logging for this cache backend:
+
+```yaml
+Flow_Mvc_Routing_Route:
+    backend: 'Flownative\RedisSentinel\RedisBackend'
+    backendOptions:
+        database: 0
+        …
+        logErrors: false
+```
+
 ## Tests
 
 You can adjust the host, port and password used in the functional tests