Skip to content

fix(s3): fallback retry with Content-MD5 for DeleteObjects#1330

Open
BorisTyshkevich wants to merge 1 commit intoAltinity:masterfrom
BorisTyshkevich:codex/fix-s3-deleteobjects-content-md5-fallback
Open

fix(s3): fallback retry with Content-MD5 for DeleteObjects#1330
BorisTyshkevich wants to merge 1 commit intoAltinity:masterfrom
BorisTyshkevich:codex/fix-s3-deleteobjects-content-md5-fallback

Conversation

@BorisTyshkevich
Copy link
Contributor

@BorisTyshkevich BorisTyshkevich commented Mar 4, 2026

Summary

  • fix regression introduced by batch DeleteObjects in 2.6.42 for S3-compatible providers requiring Content-MD5 (e.g. Huawei S3)
  • keep existing request_content_md5: true behavior unchanged
  • add helper + tests for detecting missing Content-MD5 response

Details

When DeleteObjects fails with InvalidRequest + missing Content-MD5, retry the same batch once with MD5 middleware enabled.

Validation

  • go test ./pkg/storage -run TestIsDeleteObjectsMissingContentMD5Error -v
  • go test ./pkg/storage

Closes #1329

@Slach
Copy link
Collaborator

Slach commented Mar 4, 2026

did you see d18eef7 ?

@Slach Slach closed this Mar 4, 2026
@BorisTyshkevich
Copy link
Contributor Author

yes, and already recommended to the client

@Slach Slach reopened this Mar 4, 2026
@Slach
Copy link
Collaborator

Slach commented Mar 4, 2026

ok. will merge master and will merge later

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.

Regression in 2.6.42: S3 DeleteObjects fails on Huawei S3 (missing Content-MD5)

3 participants