fix validation of redis context parameters array (#315)

[stklcode]

The optional 7th argument the Redis::connect() holds an array of context parameters, required for authentication. This parameter can either be NULL or an array.

Fix the condition, so it does not reject arrays.

resolves #315

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[lloc]

That seems OK to me. I guess this becomes more and more complex. I will do a PR and bring PHPStan in. ;-)

[stklcode]

Yes, this validation is not very pretty.

And spoiler alert: it becomes even more of a mess if we want to extend it to accept PHPRedis 6 syntax

[
    'host' => '127.0.0.1',
    'port' => 6379,
    'connectTimeout' => 2.5,
    'auth' => ['phpredis', 'phpredis'],
    'ssl' => ['verify_peer' => false],
    'backoff' => [
        'algorithm' => Redis::BACKOFF_ALGORITHM_DECORRELATED_JITTER,
        'base' => 500,
        'cap' => 750,
    ],
]

We should probably think about dropping the deep sanitization, advanced Redis config is a "pro-feature" anyway.