Compatible with aliyun redis instance

tourze · web-flow · commit c514ed41d2c0 · 2022-11-20T00:26:40.000+08:00
Some cloud provider's redis instance is just compatible in common use command, but not some special command. Just like aliyun redis instance, doc: https://help.aliyun.com/document_detail/26342.html It based on redis protocol, but not really like the redis I know... I found that `$host->info('Memory')` will return false/null sometime, so and more safe check will be better for those special redis server.
diff --git a/src/Symfony/Component/Cache/Adapter/RedisTagAwareAdapter.php b/src/Symfony/Component/Cache/Adapter/RedisTagAwareAdapter.php
@@ -292,13 +292,13 @@ private function getRedisEvictionPolicy(): string
         foreach ($hosts as $host) {
             $info = $host->info('Memory');
 
-            if ($info instanceof ErrorInterface) {
+            if ($info === false || $info === null || $info instanceof ErrorInterface) {
                 continue;
             }
 
             $info = $info['Memory'] ?? $info;
 
-            return $this->redisEvictionPolicy = $info['maxmemory_policy'];
+            return $this->redisEvictionPolicy = $info['maxmemory_policy'] ?? '';
         }
 
         return $this->redisEvictionPolicy = '';

Original file line number	Diff line number	Diff line change
`@@ -292,13 +292,13 @@ private function getRedisEvictionPolicy(): string`
`292`	`292`	`foreach ($hosts as $host) {`
`293`	`293`	`$info = $host->info('Memory');`
`294`	`294`
`295`		`- if ($info instanceof ErrorInterface) {`
	`295`	`+ if ($info === false \|\| $info === null \|\| $info instanceof ErrorInterface) {`
`296`	`296`	`continue;`
`297`	`297`	`}`
`298`	`298`
`299`	`299`	`$info = $info['Memory'] ?? $info;`
`300`	`300`
`301`		`- return $this->redisEvictionPolicy = $info['maxmemory_policy'];`
	`301`	`+ return $this->redisEvictionPolicy = $info['maxmemory_policy'] ?? '';`
`302`	`302`	`}`
`303`	`303`
`304`	`304`	`return $this->redisEvictionPolicy = '';`