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.

Sign up for GitHub

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

Jump to bottom

Consumer can not receive total messages #221

Open
weibao opened this issue Sep 2, 2020 · 3 comments · May be fixed by #220
Open

Consumer can not receive total messages #221

weibao opened this issue Sep 2, 2020 · 3 comments · May be fixed by #220

Comments

@weibao
Copy link

weibao commented Sep 2, 2020

When using RedisChannelLayer, if multiple redis hosts are set in CONFIG, some messages from outside of consumer will not be received.

Use send method, not group_send.

  • django 3.0.9
  • channels 2.4.0
  • channels-redis 3.0.1
  • daphne 2.5.0
@weibao
Copy link
Author

weibao commented Sep 3, 2020

Found the bug, replace line 300 in the core.py file of channels_redis with
index = self.consistent_hash(channel_non_local_name)

@carltongibson
Copy link
Member

Hi @weibao — Can you open an issue on https://github.com/django/channels_redis/ perhaps with a PR, and a test showing the issue?

@weibao
Copy link
Author

weibao commented Sep 3, 2020

@carltongibson Sorry, I don't know much about git and github, it takes some time to learn these.

@carltongibson carltongibson transferred this issue from django/channels Sep 3, 2020
@weibao weibao linked a pull request Sep 3, 2020 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

consistent_hash parameter in the send method is incorrect weibao/channels_redis
2 participants
@carltongibson @weibao