Progress goes straight to 100% when starting a file upload #1013
Labels
Comments
This was referenced Nov 8, 2023
gilest
added a commit
that referenced
this issue
Nov 10, 2023
Skips the browser, http, network etc. and lets us test the smallest unit of our internals – an UploadFile having its state updated by a series of ProgressEvent events. For each scenario I'm pretty sure we only need to test a single file with a stream of synchronous events. Covers some recorded events from different browsers and endpoints. We can add more scenarios here if we come across them. Specifically, the Chrome scenario proves the fix in #1016 – this was previously failing early with 100% progress, which is the same issue reported by users in #1013 I've painstakingly copied values from log screenshots to setup this existing suite. Hopefully contributors can PR full tests in future to make things a bit easier 😅 Fixes #1013
|
Fix released in 8.3.1 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Had this reported to me via Discord DM.
Suspect there is variance between the HTTP responses, timing etc of different users' upload endpoints, possibly includes browser implementation variance also.
Ref:
#1002
#1005
#1007
Suggested way forward
Upload telemetry could be gathered by adding logging to the upload function directly in
node_modulesThe text was updated successfully, but these errors were encountered: