nbren12 / geostreams

Streaming data from fortran climate/weather models.
7 stars 3 forks source link

Use same environmental variable for storing redis password, host, and port #3

Closed nbren12 closed 7 years ago

nbren12 commented 7 years ago

It is PASS in src/redis_wrapper.c and REDIS_PW in src/client.py. I think the latter is probably better. We should also probably have REDIS_URL and REDIS_PORT variables as well.

phargogh commented 7 years ago

I'll update the python files to use the environment variables REDIS_PW, REDIS_URL and REDIS_PORT instead of the defaults.

nbren12 commented 7 years ago

I also changed to using this env variables in redis_wrapper.c. I forgot which commit this was though, but I think this issue is resolved.