Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

S3 CompleteMultipartUpload requires MultipartUpload listing #2869

Open
larry-cdn77 opened this issue Sep 3, 2023 · 1 comment
Open

S3 CompleteMultipartUpload requires MultipartUpload listing #2869

larry-cdn77 opened this issue Sep 3, 2023 · 1 comment
Assignees
Labels
documentation This is a problem with documentation. service-api This issue is due to a problem in a service API, not the SDK implementation.

Comments

@larry-cdn77
Copy link

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

@larry-cdn77 larry-cdn77 added documentation This is a problem with documentation. needs-triage This issue or PR still needs to be triaged. labels Sep 3, 2023
@RanVaknin RanVaknin self-assigned this Sep 7, 2023
@RanVaknin
Copy link
Contributor

RanVaknin commented Sep 7, 2023

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

@RanVaknin RanVaknin removed the needs-triage This issue or PR still needs to be triaged. label Sep 7, 2023
@RanVaknin RanVaknin transferred this issue from aws/aws-sdk-go-v2 Sep 7, 2023
@RanVaknin RanVaknin added the service-api This issue is due to a problem in a service API, not the SDK implementation. label Sep 7, 2023
@zshzbh zshzbh transferred this issue from aws/aws-sdk Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This is a problem with documentation. service-api This issue is due to a problem in a service API, not the SDK implementation.
Projects
None yet
Development

No branches or pull requests

2 participants