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

support big files (>250 MB) #12

Open
maraoz opened this issue Jun 30, 2014 · 3 comments
Open

support big files (>250 MB) #12

maraoz opened this issue Jun 30, 2014 · 3 comments
Labels

Comments

@maraoz
Copy link
Owner

maraoz commented Jun 30, 2014

No description provided.

@maraoz maraoz added the medium label Jun 30, 2014
@kyonetca
Copy link

kyonetca commented Jul 2, 2014

Hello,
the Calculating the SHA256 digest does support files larger than 150MB as i have Hashed a Document that was 254.8 MB (254,803,968 bytes) but it still seems to fail when uploading anything over about 1.5GB to 4GB

It has to do with some files not Hashing the SHA256 Digest properly on client-side JS library crypto.js

@maraoz maraoz changed the title support big files (>150 MB) support big files (>250 MB) Jul 3, 2014
@Nd60
Copy link

Nd60 commented Feb 9, 2016

The problem is in the FileReader Javascript function.

http://stackoverflow.com/questions/20706418/are-there-file-size-limitations-when-using-javascript-filereader-api

Chrome 45, for example, does not allow files bigger than 261 MB. Sadly, e.target.result returns an empty string, without any other error.
reader.onload = function(e) {
var data = e.target.result;
....
}

This means that all files larger than 261 MB are returning e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855, which is the SHA256 hash of the empty string.

I have tried myself to stop the process if data=="" , but it works at first and in later tests it brings the unwelcome Chrome message "Aw, Snap!". So maybe it is better to just check the size of the file being evaluated first and inform the user that the maximum acceptable size is 250 MB.

@maraoz
Copy link
Owner Author

maraoz commented Feb 9, 2016

Hey @Nd60 thanks for reporting that back. It looks like that's the problem 👍

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

No branches or pull requests

3 participants