ember-cli-deploy-redis icon indicating copy to clipboard operation
ember-cli-deploy-redis copied to clipboard

Simple redis url does not work

Open alexandru-calinoiu opened this issue 5 years ago • 2 comments

Running a js app inside a container,

In deploy.js I have the following simple config

  ENV.redis = {
    url: process.env['REDIS_URL']
  }

The url in the env is REDIS_URL=redis://redis:6379/0

Apparently it will fail to parse as it will still try to connect to localhost (the default host).

To work around it I need to do the parsing myself:

  const { URL } = require('url');
  const redisURL = new URL(process.env['REDIS_URL'])

  ENV.redis = {
    host: redisURL.hostname,
    port: redisURL.port,
    username: redisURL.username,
    password: redisURL.password
  };

It will be ideal to have the parsing done by this library.

alexandru-calinoiu avatar Aug 04 '20 07:08 alexandru-calinoiu

Thanks for the report @alexandru-calinoiu.

For you or anyone else who would like to investigate, the URL is parsed by the underlying redis client, ioredis.

We do have some code in this plugin that attempts to remove any username from the URL before passing it along. This was added to support how heroku exposes its redis URLs, IIRC. That code could be the culprit.

lukemelia avatar Aug 04 '20 10:08 lukemelia

Got it, opened an issue with them, will leave this open and eventually advise to update the dependency on ioredis if they fix the issue.

alexandru-calinoiu avatar Aug 14 '20 05:08 alexandru-calinoiu