Fix potential bug, add changelog

In the source for redis, the default value for password is None, not "".
If the default is an empty string that may cause some unintended
consequences -- though it's entirely possible that Redis does not
support empty passwords, it's always safer to match the underlying
existing codebase.
This commit is contained in:
Wayne Werner 2021-06-01 15:40:52 -05:00 committed by Daniel Wozniak
parent 41fc95ab76
commit 2d1746c791

View file

@ -166,7 +166,7 @@ def _get_options(ret=None):
"port": __opts__.get("redis.port", 6379),
"unix_socket_path": __opts__.get("redis.unix_socket_path", None),
"db": __opts__.get("redis.db", "0"),
"password": __opts__.get("redis.password", ""),
"password": __opts__.get("redis.password", None),
"cluster_mode": __opts__.get("redis.cluster_mode", False),
"startup_nodes": __opts__.get("redis.cluster.startup_nodes", {}),
"skip_full_coverage_check": __opts__.get(