search

ibp-network / ibp-monitor

15 stars 10 forks source link

fix --requirepass for Redis #81

Closed paulormart closed 1 year ago

paulormart commented 1 year ago

Add a generic default password for Redis

(TODO: set a variable in docker compose so that password and users could be stored in .env file)

dcolley commented 1 year ago

Do we need to add ioredis to package.json? Another measure could simply be to remove the ports for ibp-redis from the docker-compose.yml when deploying to production. Internal network will still be available

paulormart commented 1 year ago

ioredis is one of bullmq dependencies, there is no need to add it to package.json.

I haven't tried removing the ports, or like suggesting by others if this could also be an effective option:

ports:
  - '127.0.0.1:6379:6379'