-
-
Notifications
You must be signed in to change notification settings - Fork 47
Open
Description
Problem
Following the discussion on #2927, I found a memory leak that affects both images and videos. This happens in ChatAttachmentPreview.vue, and these are the steps to reproduce (copied over from #2927):
- Sent a message with an image
- Opened that blob URL on a different tab (this works, which is great)
- Switched to the dashboard & opened a tab with that same URL (this still worked, and shouldn't have)
- Switched to the chatroom & I got a different blob URL (this is how it should be, but the other URL shouldn't be active)
The same happens with a video being shared.
Solution
Revoke URLs to prevent memory leaks. Maybe can be addressed at the same time as #2927 since that'll already require changing / refactoring the management logic for object URLs.