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.

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

"Out of Memory" when uploading files #180

Open
Erudition opened this issue Oct 27, 2023 · 2 comments
Open

"Out of Memory" when uploading files #180

Erudition opened this issue Oct 27, 2023 · 2 comments

Comments

@Erudition
Copy link

Erudition commented Oct 27, 2023

Summary

Problem

When trying to upload a video to the videos folder, It fails like this:
image

Impact

Cannot seem to upload files, or perhaps it's a file size limit? The UI does not really say what's going on. This was a 2GB video, but I also tried with some audio files that are a few MB.

Solution

A bug fix, or at least a more informative error message.

Detail

Describe the bug

01:00:54.220 out of memory 4 index.ts:256:10
    ibe index.ts:256
    im index.ts:237

To Reproduce
Open Fission Drive (in Firefox 115?)
Try to upload files.

Expected behavior
File is uploaded and added to directory.

Screenshots
image

Desktop (please complete the following information):

  • OS: Gnu Guix
  • Browser: Firefox
  • Version: 115
@Erudition Erudition changed the title Out of Memory "Out of Memory" when uploading most files Oct 27, 2023
@Erudition Erudition changed the title "Out of Memory" when uploading most files "Out of Memory" when uploading files Oct 27, 2023
@matheus23
Copy link
Member

Cannot seem to upload files, or perhaps it's a file size limit? The UI does not really say what's going on. This was a 2GB video, but I also tried with some audio files that are a few MB.

Hm. Weird. It seems to work for me in Firefox. A 2GB file will... have some issues. The way it's coded at the moment it would have to load the whole thing into memory to decrypt, but I've just tested it with a 40+MB video file and that seems to work, although it'd take some time to play.

@Erudition
Copy link
Author

Some tiny files triggered the same error, so I guess it's not about size. I notice that extra symbols in the filename can cause a crash too.
More specific error messages would help here :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants