Tim Starling has submitted this change and it was merged.

Change subject: Fixed redis auth error spam in logs.
......................................................................


Fixed redis auth error spam in logs.

Change-Id: I52e074b530d4dba2a0a8ec9b7d32f96d4527c6f4
---
M includes/clientpool/RedisConnectionPool.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Tim Starling: Verified; Looks good to me, approved



diff --git a/includes/clientpool/RedisConnectionPool.php 
b/includes/clientpool/RedisConnectionPool.php
index 8a3a1be..2d6df0e 100644
--- a/includes/clientpool/RedisConnectionPool.php
+++ b/includes/clientpool/RedisConnectionPool.php
@@ -94,7 +94,7 @@
                        $options['persistent'] = false;
                }
                if ( !isset( $options['password'] ) ) {
-                       $options['password'] = '';
+                       $options['password'] = null;
                }
                return $options;
        }

-- 
To view, visit https://gerrit.wikimedia.org/r/53290
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I52e074b530d4dba2a0a8ec9b7d32f96d4527c6f4
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Tim Starling <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to