[Lock] Fallback to eval when LOAD fails due to missing script

santysisi · santysisi · commit 54429752bb36 · 2025-07-19T16:12:12.000-03:00
diff --git a/src/Symfony/Component/Lock/Store/RedisStore.php b/src/Symfony/Component/Lock/Store/RedisStore.php
@@ -294,9 +294,14 @@ private function evaluate(string $script, string $resource, array $args): mixed
             }
         }
 
-        $this->handlePredisError(fn () => $this->redis->script('LOAD', $script));
+        try {
+            $this->handlePredisError(fn () => $this->redis->script('LOAD', $script));
 
-        return $this->handlePredisError(fn () => $this->redis->evalSha($scriptSha, 1, $resource, ...$args));
+            return $this->handlePredisError(fn () => $this->redis->evalSha($scriptSha, 1, $resource, ...$args));
+        } catch (\Predis\NotSupportedException) {
+            // If 'script LOAD' isn't supported, fallback to eval
+            return $this->redis->eval(...array_merge([$script, 1, $resource], $args));
+        }
     }
 
     private function getUniqueToken(Key $key): string

Original file line number	Diff line number	Diff line change
`@@ -294,9 +294,14 @@ private function evaluate(string $script, string $resource, array $args): mixed`
`294`	`294`	`}`
`295`	`295`	`}`
`296`	`296`
`297`		`- $this->handlePredisError(fn () => $this->redis->script('LOAD', $script));`
	`297`	`+ try {`
	`298`	`+ $this->handlePredisError(fn () => $this->redis->script('LOAD', $script));`
`298`	`299`
`299`		`- return $this->handlePredisError(fn () => $this->redis->evalSha($scriptSha, 1, $resource, ...$args));`
	`300`	`+ return $this->handlePredisError(fn () => $this->redis->evalSha($scriptSha, 1, $resource, ...$args));`
	`301`	`+ } catch (\Predis\NotSupportedException) {`
	`302`	`+ // If 'script LOAD' isn't supported, fallback to eval`
	`303`	`+ return $this->redis->eval(...array_merge([$script, 1, $resource], $args));`
	`304`	`+ }`
`300`	`305`	`}`
`301`	`306`
`302`	`307`	`private function getUniqueToken(Key $key): string`