Skip to content
New issue

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

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

Already on GitHub? # to your account

Add clean_channel method #339

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fosterseth
Copy link

@fosterseth fosterseth commented Nov 5, 2022

requires this django PR django/channels#1948

addresses memory leak issue #276

@acu192
Copy link
Collaborator

acu192 commented Nov 11, 2022

See my other comment I just made: django/channels#1948 (comment)

I do have permissions in this repo, so I ran the workflows for you.

My only comment is to update or delete this comment.

@fosterseth
Copy link
Author

@acu192 thank you for taking a look!

@carltongibson carltongibson linked an issue Nov 12, 2022 that may be closed by this pull request
@cacoos
Copy link

cacoos commented Aug 26, 2024

hey @fosterseth!

can you rebase this PR to test it? I'm having memory leaks in my project and maybe this is it!

@bigfootjon
Copy link
Collaborator

@cacosandon I've rebased the PR for the author, please test at your convenience!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Memory leak when using channels redis PubSub layer
4 participants