search

aws / aws-sdk

Landing page for the AWS SDKs on GitHub
https://aws.amazon.com/tools/
Other
68 stars 13 forks source link

S3 CompleteMultipartUpload requires MultipartUpload listing #600

Open larry-cdn77 opened 10 months ago

larry-cdn77 commented 10 months ago

Describe the issue

It took me a while to find out why my CompleteMultipartUpload is returning HTTP 400

It was because I did not list uploaded parts in the input structure under MultipartUpload

Input structures are well documented in code comments and I have come to rely on the word 'required' to decide which are the minimum input fields I have to specify

MultipartUpload does not say 'required' in the comment and would be good if it did in case it actually is a required field (as it appears to be)

Links

https://github.com/aws/aws-sdk-go-v2/blob/4f3eab1b0a3a631a4455eb7a9964c98cf0cb5536/service/s3/api_op_CompleteMultipartUpload.go#L156C19-L156C49

AWS Go SDK V2 Module Versions Used

No response

RanVaknin commented 10 months ago

Hi @larry-cdn77 ,

Thanks for raising this issue. I noticed that the S3 model from which the SDK is generated is missing the required trait for that input parameter. I have cut an internal ticket to the S3 service team in order to fix this.

Will update you when this is solved.

Thanks again Ran~

P99160076