Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

Error if a Redis TLS url is specified #196

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Strongbyte-ES
Copy link

When using a managed Redis database, a url with rediss:// is used for the Redis options, as supported by the latest ioredis.

Currently with the legacy ioredis dependency, the url with rediss:// option is ignored and the connection defaults silently to localhost:6379. The module may appear to work unintentionally if there is a localhost Redis.

This proposed change will ensure that this module "fails early" and thereby reduce debugging time.

This change would have to be removed when the ioredis dependency is upgraded e.g. to ^4.14.1

…s specified

When using a managed Redis database, a `url` with `rediss://` is used for the Redis options, as supported by the latest `ioredis.`

Currently with the legacy `ioredis` dependency, the `url` with `rediss://` option is ignored and the connection defaults silently to `localhost:6379.` The module may appear to work unintentionally if there is a localhost Redis.  

This proposed change will ensure that this module "fails early" and thereby reduce debugging time.
@Strongbyte-ES Strongbyte-ES changed the title Error if a unsupported "rediss" (Redis TLS) url is specified Error if a Redis TLS url is specified Nov 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant