As the rewrite of @uppy/aws-s3 is nearing it's completion I thought it would be best to have all the issues tracked in one place which this rewrite aims to close
I wrote regression tests ( leveraged AI heavily ) for all the issues which were reproducible and manually tested most of them against a real S3 bucket, few issues which were not entirely dependent on @uppy/aws-s3 client plugin were left out. There were total 18 open issues tagged with "AWS S3"
tests branch - https://github.com/qxprakash/uppy/tree/aws-s3-rewrite-regression-tests
rewrite base branch - https://github.com/transloadit/uppy/tree/uppy-aws-s3-rewrite
Issues fixed by rewrite with regression tests (13)
| # |
Issue |
Bug |
Regression Test |
Manual Test |
| 1 |
#3447 |
upload-success emitted multiple times per file |
✅ Pass |
✅ Verified |
| 2 |
#4313 |
Cached uploadId shared between files |
✅ Pass |
✅ Verified |
| 3 |
#4601 |
getUploadParameters called for remote files |
❌ Not Applicable |
✅ Verified |
| 4 |
#4648 |
Rapid pause/resume → TypeError on null chunk |
❌ Not Applicable |
✅ Verified |
| 5 |
#4897 |
bytesUploaded > bytesTotal |
❌ Not Applicable |
✅ Verified |
| 6 |
#4908 |
Network error → upload hangs forever |
❌ Not Applicable |
✅ Verified |
| 7 |
#5230 |
Failed uploadPart aborts multipart on S3 |
✅ Pass |
✅ Verified |
| 8 |
#5328 |
Malformed XML in completeMultipartUpload |
✅ Pass |
✅ Verified |
| 9 |
#5429 |
Prefixed keys break pause/resume |
✅ Pass |
✅ Verified |
| 10 |
#5594 |
Missing ETag prevents completion (GCS) |
✅ Pass |
❌ Not verified (needs GCS) |
| 11 |
#5667 |
XHR headers leaked into Part objects |
✅ Pass |
✅ Verified |
| 12 |
#5672 |
upload-error emitted without error object |
✅ Pass |
✅ Verified |
| 13 |
#5961 |
Golden Retriever + S3: resume hangs after refresh |
❌ Not Applicable |
✅ Verified |
Couldn't reproduce
| # |
Issue |
Bug |
Notes |
| 14 |
#3783 |
Upload hangs if computer goes to sleep |
Eliminated complex queue/retry state machines. Not unit-testable. |
Feature requests (2)
| # |
Issue |
Feature |
Status |
| 15 |
#4822 |
S3 Transfer Acceleration with STS |
✅ Implemented in #6186 |
| 16 |
#3885 |
Additional S3 checksum algorithms |
❌ Not yet addressed |
Companion / remote upload issues (3)
| # |
Issue |
Bug |
Status |
| 17 |
#5515 |
Protocol always s3-multipart for remote uploads |
❌ Not yet addressed |
| 19 |
#6181 |
Remote uploads stuck when exceeding queue limit |
⏳ in Progress |
As the rewrite of
@uppy/aws-s3is nearing it's completion I thought it would be best to have all the issues tracked in one place which this rewrite aims to closeI wrote regression tests ( leveraged AI heavily ) for all the issues which were reproducible and manually tested most of them against a real S3 bucket, few issues which were not entirely dependent on
@uppy/aws-s3client plugin were left out. There were total 18 open issues tagged with"AWS S3"tests branch - https://github.com/qxprakash/uppy/tree/aws-s3-rewrite-regression-tests
rewrite base branch - https://github.com/transloadit/uppy/tree/uppy-aws-s3-rewrite
Issues fixed by rewrite with regression tests (13)
upload-successemitted multiple times per fileuploadIdshared between filesgetUploadParameterscalled for remote filesbytesUploaded > bytesTotaluploadPartaborts multipart on S3completeMultipartUploadupload-erroremitted without error objectCouldn't reproduce
Feature requests (2)
Companion / remote upload issues (3)
s3-multipartfor remote uploads