Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update documentation for Redis Broker SSL #4812

Closed
georgepsarakis opened this issue Jun 13, 2018 · 6 comments · Fixed by #5387
Closed

Update documentation for Redis Broker SSL #4812

georgepsarakis opened this issue Jun 13, 2018 · 6 comments · Fixed by #5387

Comments

@georgepsarakis
Copy link
Contributor

According to #4792 (comment) , the documentation regarding SSL connections when Redis is used as a broker should probably be updated to include the proper configuration details.

@nmakarun
Copy link

nmakarun commented Feb 7, 2019

Hello,

Could you please help me with this issue, consumer: Cannot connect to redis: Error while reading from socket: (104, 'Connection reset by peer'),

My redis connection string looks as below,

broker_url = redis://:password@hostname:port/0

broker_use_ssl={'ssl_cert_reqs': ssl.CERT_NONE}.

I have no issues when I am using a rabbitmq backend.

@clokep
Copy link
Contributor

clokep commented Feb 7, 2019

@nmakarun I believe you have to use rediss (with an extra "s" at the end).

@nmakarun
Copy link

nmakarun commented Feb 7, 2019

@clokep I tried that as well celery is not able to recognize rediss:// and i believe it is still in development and will be available from celery 4.3 version based @georgepsarakis might know better on this.

@thedrow
Copy link
Member

thedrow commented Feb 7, 2019

I think we should consider this a blocker for 4.3 GA.
Right now we're releasing an RC so we're fine.

@nmakarun
Copy link

nmakarun commented Feb 7, 2019

@thedrow I am not sure what RC means and I am fairly new to celery.Could you please let know how to fix the url to access the azure redis with SSL ON.

@huangsam
Copy link
Contributor

@thedrow @georgepsarakis feel free to review and refine the PR I created to address docs concerning Redis Broker/Backend SSL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants