Skip to content

Commit 78f167e

Browse files
committed
With the changes in PR:sonic-net/sonic-buildimage#5289 access to redis unix socket is given to the redis group members.
Many of sonic-util commands (especially in multi-asic) case use redis unix socket to connect to DB and thus those comamnd fails without providing sudo. This PR is continuation of PR: sonic-net/sonic-buildimage#7002 where we default to use TCP for Redis if user is not root Signed-off-by: Abhishek Dosi <[email protected]>
1 parent 96c0590 commit 78f167e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/swsssdk/dbconnector.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -244,7 +244,7 @@ def __init__(self, use_unix_socket_path=False, namespace=None, decode_responses=
244244
kwargs['decode_responses'] = True
245245

246246
self.dbintf = DBInterface(**kwargs)
247-
self.use_unix_socket_path = use_unix_socket_path
247+
self.use_unix_socket_path = True if use_unix_socket_path and os.getuid() == 0 else False
248248

249249
"""If the user don't give the namespace as input, it refers to the local namespace
250250
where this application is run. (It could be a network namespace or linux host namesapce)

0 commit comments

Comments
 (0)