Support the new API GetObjectAttributes

[maeldonn]

Issue: CLDSRV-817

[bert-e]

Hello maeldonn,

My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.

Available options

name	description	privileged	authored
/after_pull_request	Wait for the given pull request id to be merged before continuing with the current one.
/bypass_author_approval	Bypass the pull request author's approval	⭐
/bypass_build_status	Bypass the build and test status	⭐
/bypass_commit_size	Bypass the check on the size of the changeset TBA	⭐
/bypass_incompatible_branch	Bypass the check on the source branch prefix	⭐
/bypass_jira_check	Bypass the Jira issue check	⭐
/bypass_peer_approval	Bypass the pull request peers' approval	⭐
/bypass_leader_approval	Bypass the pull request leaders' approval	⭐
/approve	Instruct Bert-E that the author has approved the pull request.		✍️
/create_pull_requests	Allow the creation of integration pull requests.
/create_integration_branches	Allow the creation of integration branches.
/no_octopus	Prevent Wall-E from doing any octopus merge and use multiple consecutive merge instead
/unanimity	Change review acceptance criteria from one reviewer at least to all reviewers
/wait	Instruct Bert-E not to run until further notice.

Available commands

name	description	privileged
/help	Print Bert-E's manual in the pull request.
/status	Print Bert-E's current status in the pull request TBA
/clear	Remove all comments from Bert-E from the history TBA
/retry	Re-start a fresh build TBA
/build	Re-start a fresh build TBA
/force_reset	Delete integration branches & pull requests, and restart merge process from the beginning.
/reset	Try to remove integration branches unless there are commits on them which do not appear on the source branch.

Status report is not available.

[bert-e]

Incorrect fix version

The Fix Version/s in issue CLDSRV-817 contains:

• None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

• 9.2.22

• 9.3.0

Please check the Fix Version/s of CLDSRV-817, or the target
branch of this pull request.

[codecov]

Codecov Report

❌ Patch coverage is 96.20253% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.87%. Comparing base (66a8264) to head (1f928b7).
⚠️ Report is 1 commits behind head on feature/CLDSRV-813/optional-attributes-response.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
lib/api/objectGetAttributes.js	96.82%	2 Missing ⚠️
lib/metadata/metadataUtils.js	83.33%	1 Missing ⚠️

Additional details and impacted files

Files with missing lines	Coverage Δ
lib/api/api.js	91.26% <100.00%> (+0.04%)	⬆️
lib/api/apiUtils/object/parseAttributesHeader.js	100.00% <100.00%> (ø)
lib/metadata/metadataUtils.js	94.69% <83.33%> (-0.55%)	⬇️
lib/api/objectGetAttributes.js	96.82% <96.82%> (ø)

@@                                 Coverage Diff                                 @@
##           feature/CLDSRV-813/optional-attributes-response    #6060      +/-   ##
===================================================================================
+ Coverage                                            83.79%   83.87%   +0.07%     
===================================================================================
  Files                                                  191      193       +2     
  Lines                                                12350    12429      +79     
===================================================================================
+ Hits                                                 10349    10425      +76     
- Misses                                                2001     2004       +3     

Flag	Coverage Δ
file-ft-tests	67.06% <94.93%> (+0.17%)	⬆️
kmip-ft-tests	27.01% <25.31%> (-0.02%)	⬇️
mongo-v0-ft-tests	68.37% <94.93%> (+0.18%)	⬆️
mongo-v1-ft-tests	68.36% <94.93%> (+0.16%)	⬆️
multiple-backend	34.21% <25.31%> (-0.06%)	⬇️
sur-tests	35.36% <25.31%> (+0.77%)	⬆️
sur-tests-inflights	36.42% <25.31%> (-0.07%)	⬇️
unit	69.84% <96.20%> (+0.17%)	⬆️
utapi-v2-tests	33.34% <25.31%> (-0.06%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull request overview

This pull request adds support for the AWS S3 GetObjectAttributes API, which allows clients to retrieve object metadata without downloading the object itself. The implementation includes comprehensive unit and functional tests, a new API endpoint, header parsing utilities, and updates the Arsenal dependency to include the required support for this feature.

Changes:

• Implements the GetObjectAttributes API endpoint with support for retrieving ETag, StorageClass, ObjectSize, ObjectParts, and Checksum attributes
• Adds header parsing utility to validate and parse the x-amz-object-attributes header
• Includes comprehensive unit and functional tests covering success cases, error conditions, and versioning scenarios

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
package.json	Updates Arsenal dependency to feature branch and increments version to 9.2.22
yarn.lock	Updates Arsenal lockfile entry to match feature branch reference
lib/api/objectGetAttributes.js	New API implementation for GetObjectAttributes with versioning and delete marker support
lib/api/apiUtils/object/parseAttributesHeader.js	New utility to parse and validate x-amz-object-attributes header
lib/api/api.js	Registers the new objectGetAttributes API method
constants.js	Adds allowedObjectAttributes Set defining valid attribute names
tests/unit/api/objectGetAttributes.js	Unit tests covering API functionality, error cases, and versioning
tests/unit/api/apiUtils/object/parseAttributesHeader.js	Unit tests for header parsing utility
tests/functional/aws-node-sdk/test/object/objectGetAttributes.js	Functional tests using AWS SDK

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Arsenal dependency points to a feature branch instead of a released version. The package.json and yarn.lock reference 'git+https://github.com/scality/Arsenal#feature/ARSN-549/get-object-attributes' rather than a stable release tag. Before merging this PR, the Arsenal dependency should be updated to point to a proper release version to avoid dependency management issues and ensure stability in production environments.

Suggested change

	"arsenal": "git+https://github.com/scality/Arsenal#feature/ARSN-549/get-object-attributes",
	"arsenal": "git+https://github.com/scality/Arsenal#8.2.0",

[francoisferrand]

there is already a very similar function in @DarkIsDude PR, which efficiently handles validation & parsing: should we not "merge" both code (i.e. typically make your code extend his) ?

[maeldonn]

Will be updated in another PR (process user metadata in GetObjectAttributes).

[francoisferrand]

why? it means we are duplicating the code now instead of just rebasing this PR on top of the other and handling it just once...

[maeldonn]

Because user metadata is already handled in the listObjectV2 API. That's not the case here. That would take me to an intermediate step where I should add unnecessary complexity....

Edit: Done here

[bert-e]

Incorrect fix version

The Fix Version/s in issue CLDSRV-817 contains:

• None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

• 9.2.23

• 9.3.0

Please check the Fix Version/s of CLDSRV-817, or the target
branch of this pull request.

[bert-e]

Incorrect fix version

The Fix Version/s in issue CLDSRV-817 contains:

• None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

• 9.2.24

• 9.3.0

Please check the Fix Version/s of CLDSRV-817, or the target
branch of this pull request.

[bert-e]

Incorrect fix version

The Fix Version/s in issue CLDSRV-817 contains:

• None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

• 9.2.25

• 9.3.0

Please check the Fix Version/s of CLDSRV-817, or the target
branch of this pull request.