Closed gtnx closed 1 year ago
Hello,
What do you think about this?
Hello @jrief. Could you check this PR, please? Is this ready to be merged?
Hello @jrief, love the library! I'm in the same boat as @gtnx (see: https://github.com/jrief/django-websocket-redis/issues/284). Need to use SSL in an Azure environment. Can you review this PR?
@gtnx would you like to take the maintenance of this library?
RedisPublisher
handlesssl
flagBtw, i've used black for the small parts i've changed. Have you considered using this tool. It really reduces time spent on code formatting and let developers focus on what matters.
In terms of testing, afaik travis does not support natively ssl redis-servers. Since modifications relies on well-tested code (
redis.SSLConnection
), i guess it's ok not to test it thoroughly.