muxinc / upchunk

Uploads Chunks! Takes big files, splits them up, then uploads each one with care (and PUT requests).
MIT License
337 stars 46 forks source link

`progress` event jumps backwards #30

Closed dylanjha closed 3 years ago

dylanjha commented 4 years ago

Notice in this screenshot the progress events go from 3 to 49, then jumps backwards to 33, then progresses again to 82, then jumps back to 67.

progress-evt_2020-08-17_18-00-47


In this example. my source file is 10.6 MB. At the default chunk size 5120 that will be 3 chunks. Because this backwards jump happens at 33 and 67, it seems like some funny business going with the progress event that gets emitted at the boundary of a new chunk.

Here's a video of the bug: https://stream.new/v/EBPMDDqXrxsT01ToQwBbVx2027dx5dqucOoCxkXcCMEIk

damechen commented 4 years ago

I hit the same issue. I am using upchunk@2.0.0. It also jumped back to 33 and 67.

Here is the video: https://recordit.co/dAqIMlufDc

marcelfahle commented 3 years ago

Same over here, although less apparent on bigger file sizes, where it jumps only 1% or so.

michaellimair commented 3 years ago

I tried to fix this bug, perhaps the original authors or contributors can have a look. Thanks!

https://github.com/muxinc/upchunk/pull/43

dylanjha commented 3 years ago

closed in #43 and Release 2.2.2

https://github.com/muxinc/upchunk/releases/tag/v2.2.2

thanks @michaellimair !