actumn / celery.node

Celery task queue client/worker for nodejs

Home Page:https://celery-node.js.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add "rediss://" support

dethos opened this issue · comments

Description

What is the motivation / use case for changing the behavior?

Currently the supported protocols are:

const supportedProtocols = ["redis", "amqp", "amqps"];

it would also be nice to add support for TLS connections to a redis broker and backend.

As far as I can tell, the Redis library used for this package already supports TLS, so this shouldn't be a big change

Proposed Behavior

Accept rediss:// urls for broker and backend

Thanks to @guywillett, it looks solved by #83.
Published as 0.5.9, so I'm gonna close it.
Feel free to reopen when the issue reoccurs.