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

Memory leak in yerpc #4014

Closed
Simon-Laux opened this issue Jul 5, 2024 · 0 comments · Fixed by #4023
Closed

Memory leak in yerpc #4014

Simon-Laux opened this issue Jul 5, 2024 · 0 comments · Fixed by #4023
Assignees
@Simon-Laux
Labels
bug Something isn't working new-core involves or requires an dc-node/core update

Comments

@Simon-Laux
Copy link
Member

Simon-Laux commented Jul 5, 2024 •
edited
Loading

solution: update to a version of @deltachat/jsonrpc-client that has an updated version of yerpc that includes the fix deltachat/yerpc#61

related: deltachat/yerpc#60

This leads to dc desktop crashing for a user after some hours of usage: https://support.delta.chat/t/desktop-client-on-arm-mac-freezes-after-a-few-hours/3177

Basically response promises including their result objects are not garbage collected.
@Simon-Laux Simon-Laux added the bug Something isn't working label Jul 5, 2024
@Simon-Laux Simon-Laux self-assigned this Jul 5, 2024
@Simon-Laux Simon-Laux added wait-for-core Waiting for an related core issue to be resolved new-core involves or requires an dc-node/core update and removed wait-for-core Waiting for an related core issue to be resolved labels Jul 5, 2024
@Simon-Laux Simon-Laux mentioned this issue Jul 10, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Simon-Laux Simon-Laux

Labels
bug Something isn't working new-core involves or requires an dc-node/core update
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

update core to 1.141.2 deltachat/deltachat-desktop
1 participant
@Simon-Laux