search

cameronmaske / celery-once

Celery Once allows you to prevent multiple execution and queuing of celery tasks.
https://pypi.python.org/pypi/celery_once/
BSD 2-Clause "Simplified" License
661 stars 90 forks source link

Rediss #94

Closed gustavoalmeida closed 5 years ago

gustavoalmeida commented 5 years ago

Hello all

What about urls with rediss://? We are not allowing this protocol here? https://github.com/cameronmaske/celery-once/blob/master/celery_once/backends/redis.py#L29

cameronmaske commented 5 years ago

Hi @gustavoalmeida

Didn't know rediss was a thing! It isn't excluded for a particular reason. PR welcome if you'd like to extend it to include rediss://

gustavoalmeida commented 5 years ago

Hi @cameronmaske ,

Could you please prioritize this PR #98?

Thank you

cameronmaske commented 5 years ago

@gustavoalmeida released in version 2.1.2 Thanks for the contribution!