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

Failing to Receive large files via web app #78

Open
5 tasks
donpui opened this issue Nov 22, 2022 · 3 comments
Open
5 tasks

Failing to Receive large files via web app #78

donpui opened this issue Nov 22, 2022 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@donpui
Copy link
Contributor

donpui commented Nov 22, 2022

Winden app fails (actually crash on receive side) while sending large files (500 MB, 1 GB or more) depending on user PC, Mobile device, browsers.
There is no consistency of file size which fails. Currently we are limiting to 200 MB of files in Web App.
Destiny Desktop/Mobile applications don't have this issue.

Original issue: LeastAuthority/Transfer#25
Internal bug: #OP 238

There were already few attempts to fix this memory leakage in w-w wasm:
LeastAuthority/wormhole-william#76 and web app.

Minimal client for experimentation: https://github.com/LeastAuthority/wormhole-william-minimal-client

Few more tasks, ideas to try out:

@donpui donpui added the bug Something isn't working label Nov 22, 2022
@donpui
Copy link
Contributor Author

donpui commented Dec 23, 2022

Performed some testing, as there were some refactoring, memory leak fixes.

Large File Transfer Test Cases - 2022-12-23.ods

Mostly fails receiving files on Mobile devices. Desktop is quite stable with 1 GB file transfer (send/receive). Even managed to send 5GB files.

Overall issue is on receiver side, when there is lack of memory to store all received data till full file received.

Also our current file size limitation on winden is incorrect setup. We should limit not only on sender side, but also on receiver, when file offer is presented.

@donpui
Copy link
Contributor Author

donpui commented Dec 23, 2022

@JustusFT @vu3rdd added few tasks to try out. Please put more, if something pops up.

May be we need to rethink whole process of receiving data, for example: may first get all bytes and push to temp file on system and only later on decrypt and generate downloadable file?

Also if we have mostly issues with mobile browsers, maybe we can separate limitation for desktop and mobile browser. For mobile experience, we could promote installable apps.

@donpui donpui changed the title Failing to send large files via web app Failing to Receive large files via web app Dec 23, 2022
@donpui
Copy link
Contributor Author

donpui commented Jan 30, 2023

As issue still remains on Receiver side, which very much depends on Receiver computer/phone resources and browsers memory management.

Possible short term solutions:

  • promote installable applications in Winden.app (Add labels to promote installable version app #88);
  • remove file size check for sender on upload page, but add notification, that large files might fail on receiver side if web app is used. Promote installable app for receiver (Notify sender);
  • add large file check on receiver when file offer is received:
    • if file is large then 100/200 mb. suggest to use install apps Desktop/Browsers (this could be depended from which platform he comes) and repeat upload process again, as code is already invalid;
    • notify, that download might fail. Failure most likely will happen for mobile browsers. While desktop browsers might succeed with 1GB transfers;

cc: @JustusFT @wuan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants