Skip to content

fix(s3): fix incorrect md5 for list operations#7298

Merged
Xuanwo merged 1 commit intoapache:mainfrom
dentiny:hjiang/fix-md5-list
Mar 24, 2026
Merged

fix(s3): fix incorrect md5 for list operations#7298
Xuanwo merged 1 commit intoapache:mainfrom
dentiny:hjiang/fix-md5-list

Conversation

@dentiny
Copy link
Copy Markdown
Contributor

@dentiny dentiny commented Mar 24, 2026

Which issue does this PR close?

Closes #7297

Rationale for this change

As described in the issue, we're now referring MD5 from ETag and set it to returned object metadata.
But checking doc (link attached in the issue), all involved list operations don't return MD5, which means we're returning incorrect metadata.

What changes are included in this PR?

The MD5 field in the object metadata is optional, so in this PR I simply leave it empty.

Are there any user-facing changes?

No.

AI Usage Statement

No AI involvement.

@dentiny dentiny requested a review from Xuanwo as a code owner March 24, 2026 11:28
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. releases-note/fix The PR fixes a bug or has a title that begins with "fix" labels Mar 24, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Mar 24, 2026
@Xuanwo Xuanwo merged commit cc1f90b into apache:main Mar 24, 2026
104 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/fix The PR fixes a bug or has a title that begins with "fix" size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: incorrect MD5 checksum

2 participants