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

Return 400 Bad Request if multipart upload does not contain upload parts #1304

Merged
merged 1 commit into from
Mar 18, 2016

Conversation

jadeallenx
Copy link

During a multipart upload, the "complete" message should contain a list of part ids and MD5 hashes. If it is empty, we should return a 400 Bad Request, Malformed XML message like S3. The associated test for this test is here. Addresses #1100

During a multipart upload, the "complete" message should contain
a list of part ids and MD5 hashes. If it is empty, we should
return a 400 Bad Request, Malformed XML message like S3.
@tburghart
Copy link

+1 b4491c6

borshop added a commit that referenced this pull request Mar 18, 2016
Return 400 Bad Request if multipart upload does not contain upload parts

Reviewed-by: tburghart
@tburghart
Copy link

@borshop merge

borshop added a commit that referenced this pull request Mar 18, 2016
Return 400 Bad Request if multipart upload does not contain upload parts

Reviewed-by: tburghart
@borshop borshop merged commit b4491c6 into 2.1 Mar 18, 2016
@tburghart tburghart deleted the mra/rcs-156 branch March 18, 2016 22:03
@kuenishi
Copy link
Contributor

@mrallen1 The test you mentioned is omitted because of the decorator fails_on_rcs ; https://github.com/basho/s3-tests/blob/riakcs-2.1/s3tests/functional/test_s3.py#L4599 which is written here.

@kuenishi
Copy link
Contributor

I meant this line must be removed and get this tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants