feat: redis can used persistent

Author: ddevsr

system/Cache/Handlers/RedisHandler.php

@@ -34,6 +34,7 @@ class RedisHandler extends BaseHandler
      *   password: string|null,
      *   port: int,
      *   timeout: int,
+     *   persistent: bool,
      *   database: int,
      * }
      */
@@ -42,6 +43,7 @@ class RedisHandler extends BaseHandler
         'password' => null,
         'port'     => 6379,
         'timeout'  => 0,
+        'persistent' => false,
         'database' => 0,
     ];
 
@@ -82,10 +84,11 @@ public function initialize()
         $this->redis = new Redis();
 
         try {
+            $funcConnection = isset($config['persistent']) && $config['persistent'] ? 'pconnect' : 'connect';
+
             // Note:: If Redis is your primary cache choice, and it is "offline", every page load will end up been delayed by the timeout duration.
             // I feel like some sort of temporary flag should be set, to indicate that we think Redis is "offline", allowing us to bypass the timeout for a set period of time.
-
-            if (! $this->redis->connect($config['host'], ($config['host'][0] === '/' ? 0 : $config['port']), $config['timeout'])) {
+            if (! $this->redis->$funcConnection($config['host'], ($config['host'][0] === '/' ? 0 : $config['port']), $config['timeout'])) {
                 // Note:: I'm unsure if log_message() is necessary, however I'm not 100% comfortable removing it.
                 log_message('error', 'Cache: Redis connection failed. Check your configuration.');