Merge remote-tracking branch 'github/pr/346'

This commit is contained in:
Daniele Alessandri
2016-06-14 17:19:32 +02:00
2 changed files with 5 additions and 5 deletions
@@ -239,9 +239,10 @@ class SentinelReplication implements ReplicationInterface
}
if (is_array($parameters)) {
// We unset "password" and "database" from user-supplied parameters
// as they are not needed when connecting to sentinels.
unset($parameters['database'], $parameters['password']);
// We explicitly set "database" and "password" to null,
// so that no AUTH and SELECT command is send to the sentinels.
$parameters['database'] = null;
$parameters['password'] = null;
if (!isset($parameters['timeout'])) {
$parameters['timeout'] = $this->sentinelTimeout;
@@ -44,8 +44,7 @@ class SentinelReplicationTest extends PredisTestCase
$parameters = $replication->getSentinelConnection()->getParameters()->toArray();
$this->assertArrayNotHasKey('password', $parameters);
$this->assertArrayNotHasKey('database', $parameters);
$this->assertArraySubset(array('database' => null, 'password' => null), $parameters);
}
/**