-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Frontend memory leak on client devices with livestreams #621
Comments
I'll retry on Firefox stable and Chromium. about:memory showed and error message on the process that was hosting the invidious tab. |
(meanwhile if someone else can reproduce that would be a good help) |
I expect this is videojs/video.js#5973. #235 may also be relevant. I haven't kept a tab open long enough to reproduce a crash, but I do observe constant increase in memory on Chrome 75.0.3770.100. Hopefully there's some more information that can be provided that'll be helpful for the videojs folks. I have a similar heap snapshot to the linked issue, so I believe this is the same issue: |
Well, I win the prize for more RAM used! :D I can confirm this bug. It's not the first time, and I saw it happen both on a self-hosted instance and on third parties instances. In my case I have Firefox and Archlinux. The next time it happens I will try to look into the console. |
Still relevant on Firefox: 73.0.1 and OS: Manjaro, although i don't have anything to add to what has already been mentioned. |
Is that issue still relevant? We've updated multiple times videoJS since then, so should be fixed by now. |
Probably, videojs/video.js#5973 is still opened at videojs. |
They never scheduled time 😢 |
This issue has been automatically marked as stale and will be closed in 30 days because it has not had recent activity and is much likely outdated. If you think this issue is still relevant and applicable, you just have to post a comment and it will be unmarked. |
(I forgot to reopen the tab to measure only the additional RAM after the hour of runtime)
OS: Solus
Browser: Firefox 68.0b14
The text was updated successfully, but these errors were encountered: