fix(client-s3,aws-models): change CompleteMultipartUpload to match doc

[paulreece42]

See issue #1814 , I just did a search and replace so everything is consistent:

$ find . -exec sed s/CompletedMultipartUpload/CompleteMultipartUpload/g {} \;

Issue

#1814

Description

Correctness fix as most everything uses "CompleteMultipartUpload" instead of "CompletedMultipartUpload" which does not match the docs (longer discussion in #1814 )

Testing

Ran a node script to do a multipart upload, and it worked. Full test suite was not run.

Additional context

None

---

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[youbek]

Hi Guys!

We are using the Upload method from the lib-storage package, and it's also throwing the same MalformedACLError. Because under the hood, upload method from lib-storage uses CompleteMultipartUploadCommand

[youbek]

@alexforsyth @AllanZhengYP @trivikr Any updates on this? Thank you in advance!

[ignlg]

This bug is still blocking midsize and large uploads to S3, core functionality for us. Any estimation?

[brettstack]

@alexforsyth @trivikr can we please merge this? It's a typo that's causing a bug for anyone wanting to multipart upload files.

[AllanZhengYP]

Hi @paulreece42 @ignlg @brettstack

Anything in clients folder is generated, so this fix will be overwritten in the next release. This issue should be fixed by the client generator. I will create a separate fix for it as I mentioned in #1814 (comment). Close it for now.

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.