Skip to content

Commit

Permalink
Merge pull request #38568 from rawtaz/fix-redis-empty-password
Browse files Browse the repository at this point in the history
redis: Do not try to authenticate with non-string password/user
  • Loading branch information
icewind1991 authored Jun 1, 2023
2 parents ff66104 + c52aaa3 commit e81fdfe
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/private/RedisFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -72,8 +72,8 @@ private function create() {
}

$auth = null;
if (isset($config['password']) && $config['password'] !== '') {
if (isset($config['user']) && $config['user'] !== '') {
if (isset($config['password']) && (string)$config['password'] !== '') {
if (isset($config['user']) && (string)$config['user'] !== '') {
$auth = [$config['user'], $config['password']];
} else {
$auth = $config['password'];
Expand Down

0 comments on commit e81fdfe

Please sign in to comment.