[12.x] - Redis - Establish connection first, before set the options

[alexmontoanelli]

Hi,

PhpRedisConnector try to set 4 options (max_retries,backoff_algorithm,backoff_base,backoff_cap) before
the connection has been established, and so, the options has no effect at all.

With a single line move, creating the connection/client before, the options then is set successfully on client.

[taylorotwell]

Don't these values specifically control the amount of retries when connecting though?

[alexmontoanelli]

These values are passed to client, in this case \Redis, to controll retries and backoff strategies.

If you set in your database.conf this settings (max_retries, backoff_algorithm), and then run
this code, you get the default values.

` php
$connection = Redis::connection();
$option1 = $connection->client()->getOption(\Redis::OPT_MAX_RETRIES);
$option2 = $connection->client()->getOption(\Redis::OPT_BACKOFF_ALGORITHM);
dump($option1, $option2);

`

//database settings

//code to get driver settings

//return values, before and after the fix
//10,0 (default options for max retries, and backoff), event (with database configure)
//30, 2 (respecting the config file)