Merge pull request #733 from gillytech/v7

Geolim4 · web-flow · commit 406e59e31b50 · 2020-04-13T13:36:12.000Z
Removing path check in Redis driver before auth.
diff --git a/lib/Phpfastcache/Drivers/Redis/Driver.php b/lib/Phpfastcache/Drivers/Redis/Driver.php
@@ -85,10 +85,9 @@ protected function driverConnect(): bool
             $this->instance->setOption(RedisClient::OPT_PREFIX, $this->getConfig()->getOptPrefix());
         }
 
-        if (!$this->getConfig()->getPath()) {
-            if ($this->getConfig()->getPassword() && !$this->instance->auth($this->getConfig()->getPassword())) {
-                return false;
-            }
+        
+        if ($this->getConfig()->getPassword() && !$this->instance->auth($this->getConfig()->getPassword())) {
+            return false;
         }
 
         if ($this->getConfig()->getDatabase() !== null) {

Original file line number	Diff line number	Diff line change
`@@ -85,10 +85,9 @@ protected function driverConnect(): bool`
`85`	`85`	`$this->instance->setOption(RedisClient::OPT_PREFIX, $this->getConfig()->getOptPrefix());`
`86`	`86`	`}`
`87`	`87`
`88`		`- if (!$this->getConfig()->getPath()) {`
`89`		`- if ($this->getConfig()->getPassword() && !$this->instance->auth($this->getConfig()->getPassword())) {`
`90`		`- return false;`
`91`		`- }`
	`88`	`+`
	`89`	`+ if ($this->getConfig()->getPassword() && !$this->instance->auth($this->getConfig()->getPassword())) {`
	`90`	`+ return false;`
`92`	`91`	`}`
`93`	`92`
`94`	`93`	`if ($this->getConfig()->getDatabase() !== null) {`