-
Notifications
You must be signed in to change notification settings - Fork 275
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
Slow memory leak in dealer.py #2096
Comments
DZabavchik
added a commit
to NinjaRMM/crossbar
that referenced
this issue
Sep 4, 2023
DZabavchik
added a commit
to NinjaRMM/crossbar
that referenced
this issue
Sep 4, 2023
(cherry picked from commit 2b8ae44)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Finally found this leak that was haunting for 7+ years
crossbar/crossbar/router/dealer.py
Lines 1135 to 1139 in 0089c1e
Zombie invocations remain in
_invocations_by_call
with references to callee, caller, authorization that will not be GC'dTesting changes that preserve original caller session ID in
InvocationRequest
object, so it can be deleted by key from_invocations_by_call
withdel self._invocations_by_call[invocation_request.caller_session_id, invocation_request.call.request]
Will submit PR shortly
The text was updated successfully, but these errors were encountered: