Skip to content

Commit 653bdba

Browse files
authored
Merge pull request #120 from abdosi/sudo_access
What I did: Changes to Use Redis Unix Socket if the user is root else default to TCP Why I did: With the changes in PR:sonic-net/sonic-buildimage#5289 access to redis unix socket is given to the redis group members or to the root. 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 in sonic-cfggen. This should fix: sonic-net/sonic-buildimage#8501
2 parents 96c0590 + 78f167e commit 653bdba

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)