search

matus-chochlik / ctcache

Cache for clang-tidy static analysis results
Boost Software License 1.0
83 stars 29 forks source link

Constrain commands to the Redis server to the bare minimum: AUTH, GET, SET #68

Closed kaespi closed 2 months ago

kaespi commented 2 months ago

In the server configuration we use, the Redis user used to read and write the ctcache stuff has only minimal permissions. By default redis-py always sends some CLIENT SETINFO ... commands to the server. In our case this results in errors. I think these commands can be avoid in case of ctcache.

Of course, this could be made configurable, but I guess it's favorable to not overwhelm the user by configuration options. Therefore, I think this would make a good default.

matus-chochlik commented 2 months ago

Merged, thanks for the PR