fix(phpredis): honor initial read timeout (#764)

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: Gabriel Ostrolucký <gabriel.ostrolucky@gmail.com>

Author: 3 people

src/Factory/PhpredisClientFactory.php

@@ -346,8 +346,8 @@ private function createClient(RedisDsn $dsn, string $class, string $alias, array
             $dsn->getPort(),
             $options['connection_timeout'],
             $persistentId,
-            5, // retry interval
-            5, // read timeout
+            (int) ($options['read_write_timeout'] ?? 5), // retry interval
+            $options['read_write_timeout'] ?? 5, // read timeout
             $context,
         ];
 

tests/Factory/PhpredisClientFactoryTest.php

@@ -534,6 +534,10 @@ public function testMethodWithPassByRefArgument(): void
 
     public function testCreateWithConnectionPersistentTrue(): void
     {
+        $this->logger->method('debug')->with(...$this->withConsecutive(
+            ['Executing command "PCONNECT localhost 6379 5 default 1 1.5"'],
+        ));
+
         $factory = new PhpredisClientFactory(new RedisCallInterceptor($this->redisLogger));
 
         $client = $factory->create(
@@ -542,9 +546,10 @@ public function testCreateWithConnectionPersistentTrue(): void
             [
                 'connection_timeout' => 5,
                 'connection_persistent' => true,
+                'read_write_timeout' => 1.5,
             ],
             'default',
-            false,
+            true,
         );
 
         $this->assertInstanceOf(Redis::class, $client);
@@ -572,6 +577,10 @@ public function testCreateWithConnectionPersistentString(): void
 
     public function testCreateWithConnectionPersistentFalse(): void
     {
+        $this->logger->method('debug')->with(...$this->withConsecutive(
+            ['Executing command "CONNECT localhost 6379 5 <null> 1 1.5"'],
+        ));
+
         $factory = new PhpredisClientFactory(new RedisCallInterceptor($this->redisLogger));
 
         $client = $factory->create(
@@ -580,9 +589,10 @@ public function testCreateWithConnectionPersistentFalse(): void
             [
                 'connection_timeout' => 5,
                 'connection_persistent' => false,
+                'read_write_timeout' => 1.5,
             ],
             'default',
-            false,
+            true,
         );
 
         $this->assertInstanceOf(Redis::class, $client);