Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Friendly Remind #3

Open
LYNCSS opened this issue Jun 20, 2019 · 0 comments
Open

Friendly Remind #3

LYNCSS opened this issue Jun 20, 2019 · 0 comments

Comments

@LYNCSS
Copy link

LYNCSS commented Jun 20, 2019

Hello, I am doing the same project as yours, I discovered a little bug in OmniApi script.
In the repository, it showed that you want to separate OmniApi and Redis apart. The script config.py is fundamentally had to be replaced.

However, in the script cacher.py, it separated remote and local redis server. The Api scripts that imported cacher.py always used the localhost redis client operations. Hence, if you want api scripts connect to the redis server in the docker network, you have to change the code line:
l = rredis.StrictRedis() into
l = redis.StrictRedis(host=config.REDIS_HOST, port=config.REDIS_PORT, db=config.REDIS_DB)

It should work fine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant