Rename CreateMultipartUploadOutput to InitiateMultipartUploadResult #8032
Conversation
The response to a CreateMultipartUpload request should contain a root element InitiateMultipartUploadResult. This will align with the AWS API docs and with client libraries that check the root element.
There was a problem hiding this comment.
Welcome to LocalStack! Thanks for raising your first Pull Request and landing in your contributions. Our team will reach out with any reviews or feedbacks that we have shortly. We recommend joining our Slack Community and share your PR on the #community channel to share your contributions with us. Please make sure you are following our contributing guidelines and our Code of Conduct.
|
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
|
Hi @pguillory and thanks for your contribution! However, the code you are updating is auto-generated and updated every week, based on the botocore specs. Here are some examples on how we've already updated some responses: #8021, #7983 and #7545. I can also take a stab a it tomorrow if you'd like, as it is a bit tedious and also quite urgent. I'm currently planning to try to collect all XML responses from every operation to compare them to the specs and correct all of the remaining ones we are missing. |
|
There's now a PR which would fix all S3 operations root tag at #8037. |
The response to a CreateMultipartUpload request should contain a root element InitiateMultipartUploadResult. This will align with the AWS API docs and with client libraries that check the root element.
#8030