Skip to content

HDDS-15645. ListObjectsV2 returns Owner when FetchOwner is not requested - #10897

Merged
Gargi-jais11 merged 2 commits into
apache:masterfrom
sravani-revuri:HDDS-15645
Aug 3, 2026
Merged

HDDS-15645. ListObjectsV2 returns Owner when FetchOwner is not requested#10897
Gargi-jais11 merged 2 commits into
apache:masterfrom
sravani-revuri:HDDS-15645

Conversation

@sravani-revuri

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

The S3 compatibility tests test_bucket_listv2_fetchowner_defaultempty and test_bucket_listv2_fetchowner_empty are failing because in the S3 ListObjectsV2 response, the Owner field is currently included for each object even when the fetch-owner parameter is not set or is false. According to S3 behavior, owner information should only be returned when fetch-owner=true is requested. Update the response generation logic to omit the Owner field unless the fetch-owner flag is enabled.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-15645

How was this patch tested?

Written unit tests.

@sravani-revuri
sravani-revuri marked this pull request as ready for review July 29, 2026 15:07

@Gargi-jais11 Gargi-jais11 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @sravani-revuri for working on this. Overall LGTM!
Please add integration test as well in AbstractS3SDKV1Tests and AbstractS3SDKV2Tests.

@Gargi-jais11 Gargi-jais11 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @sravani-revuri for updating the patch. LGTM!, +1.
@sreejasahithi would u like to take a look on this.

@sreejasahithi sreejasahithi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @sravani-revuri for the PR.
The change LGTM.

FYI: now ListObjectsV2 responses no longer include per-object owner by default (previously always included it). Users that need owner metadata on V2 lists should pass fetch-owner=true.

@Gargi-jais11

Copy link
Copy Markdown
Contributor

Thanks @sravani-revuri for the PR. The change LGTM.

FYI: now ListObjectsV2 responses no longer include per-object owner by default (previously always included it). Users that need owner metadata on V2 lists should pass fetch-owner=true.

Yaa that's correct for per-object owner user needs list-objectv1 or fetch-owner=true.

@Gargi-jais11

Copy link
Copy Markdown
Contributor

Thanks @sravani-revuri for working on this and @sreejasahithi for the review.

@Gargi-jais11
Gargi-jais11 merged commit cf8ae81 into apache:master Aug 3, 2026
45 checks passed
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.

3 participants