saltstack/salt
1
0
Fork 0
mirror of https://github.com/saltstack/salt.git synced 2025-04-17 10:10:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

Fix potential bug, add changelog

Browse source 
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 13996cea60
commit 8dc4160a6a
2 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
changelog/58044.added Normal file
View file

@ -0,0 +1 @@
Added password support to Redis returner.

2
salt/returners/redis_return.py
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(