feature request: allow connection url

[iwpnd]

Hey @ejhayes 👋

any chance you could offer the option to pass a redis url instead of the host/port? that would allow to connect optionally via tls using rediss:// or redis://.

Looks like @nestjs/redis is supporting it here. As for implementation one could prioritise an env such as REDIS_URL over REDIS_HOST/REDIS_PORT/REDIS_PASSWORD if set.

wdyt?

[iwpnd]

optionally i believe adding tls: true would also work.

[ejhayes]

Good idea @iwpnd. I can definitely add this in.

[iwpnd]

i have a working first draft, where whenever a REDIS_URL is present in the configService I change connection parameters.

  private getConnection() {
    const hasUrl = this.configService.config.REDIS_URL !== '';

    return !hasUrl
      ? {
          connection: {
            host: this.configService.config.REDIS_HOST,
            port: this.configService.config.REDIS_PORT,
            password: this.configService.config.REDIS_PASSWORD,
          },
        }
      : {
          connection: new Redis(this.configService.config.REDIS_URL, {
            maxRetriesPerRequest: null,
          }),
        };
  }

// used like

  create(queuePrefix: string, queueName: string, queue: Queue) {
    const labels = {
      [LABEL_NAMES.QUEUE_PREFIX]: queuePrefix,
      [LABEL_NAMES.QUEUE_NAME]: queueName,
    };
    const queueEvents = new QueueEvents(queueName, {
      prefix: queuePrefix,
      ...this.getConnection(),
    });
    [...]

I understand that everybody has their way of working, so I understand that this is not the desired way to do it - yet, with some guidance I can draft a PR and complete this myself if you'd allow me to.

wdyt @ejhayes ?

[cahlan]

adding my vote for this as well. as I understand it, this package won't work with redis instances that are configured to use TLS?

[iwpnd]

This is correct @cahlan