search

ZeitOnline / celery_longterm_scheduler

Schedules celery tasks to run in the potentially far future
BSD 3-Clause "New" or "Revised" License
27 stars 8 forks source link

Supporting rediss:// scheme #8

Open mauriciomdb opened 1 year ago

mauriciomdb commented 1 year ago

Hi team! thanks for this great package.

We are trying the package and we faced a small issue when using it with a Redis encrypted (TLS) transport. The Redis connection URL when encrypted is "rediss://" instead of "redis://" and that causes the backend to fail.

We patched it in a fork in the backend.py and it worked:

BACKENDS = { 'memory': MemoryBackend, 'redis': RedisBackend, 'rediss': RedisBackend, }

It's just about supporting that "rediss" scheme.

It would be great to have this solved in the 1.2.1. :)

dbartenstein commented 7 months ago

Hi all! Isn’t that solved already solved with version 1.3? https://github.com/ZeitOnline/celery_longterm_scheduler/commit/af749962cddabb3dfc5043d36e7633a5d5ac16a1? If yes, the issue could be closed.