Fix file size on larger multipart uploads

[lehni]

We are encountering this problem with larger file uploads ,e.g. video files of roughly 20MB size. The total is missing, but the last event received has the actual file-size in the loaded property. This simple fix addresses this situation.

Fixes #183

[tcastelli]

any chances of getting this merged?

[lcgiry]

Hi,
Yes merging it would solve some issue we're facing, that would be nice !
Thanks

[immad-imtiaz]

@LinusU @anacronw Can this be merged and released?

[lehni]

Once it's merged, you can then also close #197 and #191.

[keshav-c]

+1 to see this merged 🙏

[Shandur]

up @LinusU @anacronw

[santiq]

Can you guys take a look @LinusU @anacronw ? I'm willing to add unit test for this if needed

[toritsejuFO]

I was just about to created a PR for this before seeing this PR as a suggestion to one of the issues.

Although I had a different thought towards it below

var currentSize = 0
var bytesLoaded = 0
...
...
...
upload.on('httpUploadProgress', function (ev) {
    if (ev.total) currentSize = ev.total
    if (ev.loaded) bytesLoaded = ev.loaded
})

cb(null, {
    size: currentSize,
    bytesUploaded: bytesLoaded,
    bucket: opts.bucket,
...

Nonetheless, +1 to see this merged 👍🏿, it's over a year now.

[lehni]

@toritsejuFO if that works then your code seems more correct?

[toritsejuFO]

They're the same thing. Later went with installing your commit directly. Thanks

[santiq]

I ended up forking this lib, and applying almost all of the different fixes and improvements like

#204 #216 #206 #201 #199 #191