Huge file size reported incorrect

fmisa / gwtupload

Automatically exported from code.google.com/p/gwtupload

Other

0 stars 0 forks source link

Huge file size reported incorrect #173

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

What steps will reproduce the problem?
1. Upload a file with size greater than 2Gb
2. Progress bar will report 0% (totalBytes == -1)

What version of the product are you using? On what operating system?
0.6.6

Please provide any additional information below.
The 2nd param of gwtupload.server.UploadListener.UploadListener(int, int) 
should be long.

Content length should be queried like: (see 
http://stackoverflow.com/questions/7423285/javax-servlet-httpservletrequest-getc
ontentlength-returns-int-only)
try {
size = Long.parseLong(request.getHeader("Content-Length"));
} catch (NumberFormatException e) { // cannot help
}

Original issue reported on code.google.com by pihentagy on 4 Mar 2013 at 9:55

GoogleCodeExporter commented 8 years ago

Here is a fix, don't know how to make a pull request.

http://code.google.com/r/pihentagy-gwtupload/

After applying this fileupload fix, it works for me.

https://github.com/pihentagy/commons-fileupload/commit/16a677dd3c61acde530a5f54a
59402faed1a953a

Original comment by pihentagy on 5 Mar 2013 at 3:48

GoogleCodeExporter commented 8 years ago

This is a bug in the apache commons-fileupload library gwt-upload relies on. 
We cannot do anything here, you should report the issue to them.

Anyway, thank you for reporting, this can be useful for other users 
experimenting the problem.

- Manolo

Original comment by manuel.carrasco.m on 5 Apr 2013 at 10:06

Changed state: WontFix

GoogleCodeExporter commented 8 years ago

This issue was closed by revision 017d5a04d32f.

Original comment by manuel.carrasco.m on 5 Apr 2013 at 10:20

Changed state: Fixed

GoogleCodeExporter commented 8 years ago

@pihentagy you were right there is a workaround to do in gwt-upload, fixed in 
trunk

Thanks
- Manolo

Original comment by manuel.carrasco.m on 5 Apr 2013 at 10:21

GoogleCodeExporter commented 8 years ago

FYI: apache-commons 1.3 will fix on apache side. 
https://issues.apache.org/jira/browse/FILEUPLOAD-195

Original comment by pihentagy on 5 Apr 2013 at 11:26

GoogleCodeExporter commented 8 years ago

Yes, I'll maintain our fix right now until we update the file-upload 
dependency, we have to check if it works ok with gwt-upload (I believe it does 
though)

Original comment by manuel.carrasco.m on 5 Apr 2013 at 11:58

GoogleCodeExporter commented 8 years ago

Original comment by manuel.carrasco.m on 31 Dec 2013 at 3:11

Added labels: Milestone-1.0.0

GoogleCodeExporter commented 8 years ago

Does the new tagging mean I could expect a release in the near future?

Original comment by kontra.g...@gmail.com on 31 Dec 2013 at 3:40

GoogleCodeExporter commented 8 years ago

Yes gwtupload 1.0.0 is out in maven repos.

Original comment by manuel.carrasco.m on 2 Jan 2014 at 8:38

GoogleCodeExporter commented 8 years ago

Great news, thank you.

Original comment by kontra.g...@gmail.com on 2 Jan 2014 at 8:53