From 8a0c20b54c4da311c5936629a451bf2826fe6a18 Mon Sep 17 00:00:00 2001
From: Naz <hi@nazavo.com>
Date: Tue, 14 Feb 2023 21:38:08 +0800
Subject: [PATCH] Fixed Redis cluster ttl configuration

refs https://github.com/TryGhost/Toolbox/issues/520

- The cluster config is taking over local adapter ttl configuration - the priority should be reverse adapter config first followed by cluster config
- In addition if we add nested config merging to adapter manager we could achieve the same effect by having per-adapter configuraiton with "clusterConfig.options.ttl" value
---
 ghost/adapter-cache-redis/lib/adapter-cache-redis.js | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/ghost/adapter-cache-redis/lib/adapter-cache-redis.js b/ghost/adapter-cache-redis/lib/adapter-cache-redis.js
index 1363ea19b7..f9c0966d6b 100644
--- a/ghost/adapter-cache-redis/lib/adapter-cache-redis.js
+++ b/ghost/adapter-cache-redis/lib/adapter-cache-redis.js
@@ -22,6 +22,17 @@ class Redis extends BaseCacheAdapter {
         this.cache = config.cache;
 
         if (!this.cache) {
+            // @NOTE: this condition can be avoided if we add merging of nested options
+            //        to adapter configuration. Than adding adapter-specific {clusterConfig: {options: {ttl: XXX}}}
+            //        will be enough to set ttl for redis cluster.
+            if (config.ttl && config.clusterConfig) {
+                if (!config.clusterConfig.options) {
+                    config.clusterConfig.options = {};
+                }
+
+                config.clusterConfig.options.ttl = config.ttl;
+            }
+
             this.cache = cacheManager.caching({
                 store: redisStore,
                 ttl: config.ttl,