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

Files under 16Kb are not downloaded correctly #1077

Closed
Angelmmiguel opened this issue Mar 13, 2017 · 4 comments
Closed

Files under 16Kb are not downloaded correctly #1077

Angelmmiguel opened this issue Mar 13, 2017 · 4 comments
Labels
bug

Comments

@Angelmmiguel
Copy link

@Angelmmiguel Angelmmiguel commented Mar 13, 2017

What version of WebTorrent?
0.98.10

What browser and version? (if using WebTorrent in the browser)
Google Chrome v56.0.2924.87 (64-bit)

What did you expect to happen?
I seeded several files. The size of every of them didn't exceed 16Kb. After that, I got the magnet URI and tried to download from other window of the same browser. It seems the files was dowloaded but I couldn't get the content of those files.

To double check it, I uploaded the same file to instant.io and the result was the same.

However, If I use a file > 16Kb it works fine in my implementation and instant.io. You can use the following image to check the problem:

small image

What actually happened?
The file was downloaded but I couldn't get the content of the image or text. I tried with the method appendTo and I got an empty box without any content.

Investigation
I investigate a bit the error and the _notify method of file-stream.js is only called once. The method returns on file-stream.js#L53 and never execute the get method in file-stream.js#L60.

@feross

This comment has been minimized.

Copy link
Member

@feross feross commented Mar 13, 2017

Confirmed this is an issue. Found the bug this was introduced in, using git bisect: 6ef2785

@feross feross added the bug label Mar 13, 2017
@feross

This comment has been minimized.

Copy link
Member

@feross feross commented Mar 13, 2017

Fixed in 0.98.12. Thanks for reporting!

@feross feross closed this in 9c3de0e Mar 13, 2017
@Angelmmiguel

This comment has been minimized.

Copy link
Author

@Angelmmiguel Angelmmiguel commented Mar 14, 2017

That's great! Thank you.

@lock

This comment has been minimized.

Copy link

@lock lock bot commented May 3, 2018

This thread has been automatically locked because it has not had recent activity. To discuss futher, please open a new issue.

@lock lock bot locked as resolved and limited conversation to collaborators May 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.