-
Notifications
You must be signed in to change notification settings - Fork 268
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
Are there max requests settings for Daphne? #403
Comments
Not currently, but it might be a good addition. 🤔 This is ref: Were you able to investigate why you're seeing a leak there? Also did it stabilise at 4%? (In which case it may be the Python GC rather than a leak per se — for which it'd keep going up.) |
I created this draft PR #405 Please let me know if this is of interest. I'll polish the code and make some unit tests 👍 |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
The PR in #405 adds a shutdown, but it doesn't reload. For that we'd need something like Django autoreloader's If someone wanted to take that on, it would be worth a look. |
I have this ASGI applicate with Django and Daphne, and I'm using the following command.
daphne -b 0.0.0.0 -p $PORT server.asgi:application
Is there a way to add
--max-requests
and--max-requests-jitter
to actively prevent memory leaks?Maybe something like:
daphne -b 0.0.0.0 -p $PORT --max-requests 1000 --max-requests-jitter 50 server.asgi:application
The text was updated successfully, but these errors were encountered: