[Lock] compatiblity with redis cluster 7

[bastnic]

Q	A
Branch?	5.4
Bug fix?	yes
New feature?	no
Deprecations?	no
License	MIT

While upgrading from redis 5 (no cluster) to AWS Elasticache redis 7 (cluster serverless), I stumble upon a fail.

This Redis command is not allowed from script script

Investigating, the fail is here, TIME is not available in cluster mode.

symfony/src/Symfony/Component/Lock/Store/RedisStore.php

Lines 272 to 281 in d508d5f

	if (!isset($this->supportTime)) {
	// Redis < 5.0 does not support TIME (not deterministic) in script.
	// https://redis.io/commands/eval#replicating-commands-instead-of-scripts
	// This code asserts TIME can be use, otherwise will fallback to a timestamp generated by the PHP process.
	$script = '
	local now = redis.call("TIME")
	redis.call("SET", KEYS[1], "1", "PX", 1)
	return 1
	';

redis > EVAL 'local now = redis.call("TIME"); redis.call("SET", KEYS[1], "1", "PX", 1); return 1;' 1 pouet
(error) ERR This Redis command is not allowed from script script: b0e701c57bb0187cbd6ec8e404a7f6ba19b11a94, on @user_script:1.

BUT the error does not contains commands not allowed after non deterministic anymore, so no fallback on the PHP implementation.

Hence, the new exception catch.

See https://redis.io/docs/interact/programmability/eval-intro/#replicating-commands-instead-of-scripts

[nicolas-grekas]

Thank you @bastnic.