Support the new API GetObjectAttributes by maeldonn · Pull Request #6060 · scality/cloudserver

maeldonn · 2026-01-26T09:02:05Z

bert-e · 2026-01-26T09:02:08Z

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 · 2026-01-26T09:02:14Z

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 · 2026-01-26T09:02:30Z

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
8184	1	8183	0

View the full list of 1 ❄️ flaky test(s)

"before all" hook for "should delete a object uploaded in parts successfully"::DELETE object With default signature with multipart upload "before all" hook for "should delete a object uploaded in parts successfully"

Flake rate in main: 100.00% (Passed 0 times, Failed 3 times)

Stack Traces | 0.194s run time

socket hang up

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}

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.

lib/api/objectGetAttributes.js

Copilot · 2026-01-26T09:08:49Z

package.json

    "@azure/storage-blob": "^12.28.0",
    "@hapi/joi": "^17.1.1",
-    "arsenal": "git+https://github.com/scality/Arsenal#8.2.43",
+    "arsenal": "git+https://github.com/scality/Arsenal#feature/ARSN-549/get-object-attributes",


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",

I guess you can bump ?

I'm waiting for all the reviews first. I will do it just before merging the PR. That's why I haven't resolved this comment 😉

lib/api/objectGetAttributes.js

tests/functional/aws-node-sdk/test/object/objectGetAttributes.js

constants.js

lib/api/objectGetAttributes.js

lib/api/apiUtils/object/parseAttributesHeader.js

tests/unit/api/objectGetAttributes.js

lib/api/objectGetAttributes.js

tests/functional/aws-node-sdk/test/object/objectGetAttributes.js

bert-e · 2026-01-28T10:16:56Z

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.

constants.js

lib/api/apiUtils/object/parseAttributesHeader.js

lib/api/objectGetAttributes.js

tests/functional/aws-node-sdk/test/object/objectGetAttributes.js

tests/functional/aws-node-sdk/test/versioning/objectGetAttributes.js

tests/unit/api/objectGetAttributes.js

bert-e · 2026-01-29T12:13:51Z

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 · 2026-01-30T14:54:45Z

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.

lib/api/objectGetAttributes.js

lib/metadata/metadataUtils.js

bert-e · 2026-02-12T15:00:59Z

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.26
9.3.2

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

bert-e · 2026-02-12T15:01:15Z

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.3.2

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

lib/api/objectGetAttributes.js

Issue: CLDSRV-817

bert-e · 2026-02-18T13:42:31Z

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
2 peers

The following reviewers are expecting changes from the author, or must review again:

@francoisferrand

maeldonn · 2026-02-18T14:00:39Z

/approve

bert-e · 2026-02-18T14:00:49Z

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
2 peers

The following reviewers are expecting changes from the author, or must review again:

@francoisferrand

The following options are set: approve

francoisferrand · 2026-02-18T16:22:50Z

lib/api/objectGetAttributes.js

@@ -0,0 +1,170 @@
+const { promisify } = require('util');


nit: just noticed this is indented with 2 spaces only, while we use 4-space indent overall...

Yes used 2 space indentation on new files as it described here https://scality.atlassian.net/wiki/spaces/OS/pages/3726508042/Formatting+baseline+for+our+JS+TS+projects

bert-e · 2026-02-18T16:26:03Z

I have successfully merged the changeset of this pull request
into targetted development branches:

✔️ development/9.3

The following branches have NOT changed:

development/7.10
development/7.4
development/7.70
development/8.8
development/9.0
development/9.1
development/9.2

Please check the status of the associated issue CLDSRV-817.

Goodbye maeldonn.

The following options are set: approve

maeldonn requested review from a team, DarkIsDude, Copilot and delthas January 26, 2026 09:02

Copilot started reviewing on behalf of maeldonn January 26, 2026 09:02 View session

maeldonn force-pushed the feature/CLDSRV-817/get-object-attributes branch from a7ad5b2 to 023e610 Compare January 26, 2026 09:08

Copilot AI reviewed Jan 26, 2026

View reviewed changes

maeldonn commented Jan 26, 2026

View reviewed changes

constants.js Outdated Show resolved Hide resolved

SylvainSenechal reviewed Jan 26, 2026

View reviewed changes

lib/api/objectGetAttributes.js Outdated Show resolved Hide resolved

delthas removed their request for review January 26, 2026 11:10

maeldonn commented Jan 26, 2026

View reviewed changes

lib/api/objectGetAttributes.js Outdated Show resolved Hide resolved

maeldonn marked this pull request as draft January 26, 2026 13:58

maeldonn removed the request for review from DarkIsDude January 26, 2026 16:11

francoisferrand requested changes Jan 26, 2026

View reviewed changes

maeldonn force-pushed the feature/CLDSRV-817/get-object-attributes branch from d9b4211 to 3b7dddd Compare January 27, 2026 11:01

maeldonn marked this pull request as ready for review January 27, 2026 13:14

maeldonn requested review from DarkIsDude, SylvainSenechal and francoisferrand January 27, 2026 13:14

maeldonn force-pushed the feature/CLDSRV-817/get-object-attributes branch 2 times, most recently from 5a3c013 to 23f1c1c Compare January 28, 2026 10:16

DarkIsDude requested changes Jan 28, 2026

View reviewed changes

maeldonn requested a review from DarkIsDude January 28, 2026 17:50

maeldonn force-pushed the feature/CLDSRV-817/get-object-attributes branch from c45c894 to 838227a Compare January 30, 2026 15:04

francoisferrand reviewed Feb 11, 2026

View reviewed changes

lib/api/objectGetAttributes.js Show resolved Hide resolved

francoisferrand reviewed Feb 11, 2026

View reviewed changes

lib/api/objectGetAttributes.js Outdated Show resolved Hide resolved

lib/metadata/metadataUtils.js Outdated Show resolved Hide resolved

maeldonn force-pushed the feature/CLDSRV-817/get-object-attributes branch 3 times, most recently from c762cd8 to bef8034 Compare February 12, 2026 15:00

maeldonn changed the base branch from development/9.1 to development/9.2 February 12, 2026 15:00

maeldonn changed the base branch from development/9.2 to development/9.3 February 12, 2026 15:01

maeldonn requested a review from francoisferrand February 12, 2026 20:41

maeldonn force-pushed the feature/CLDSRV-817/get-object-attributes branch from 3ebd2ab to e118699 Compare February 13, 2026 14:21

francoisferrand reviewed Feb 17, 2026

View reviewed changes

lib/api/objectGetAttributes.js Show resolved Hide resolved

lib/api/objectGetAttributes.js Show resolved Hide resolved

francoisferrand reviewed Feb 17, 2026

View reviewed changes

lib/api/objectGetAttributes.js Outdated Show resolved Hide resolved

francoisferrand reviewed Feb 17, 2026

View reviewed changes

lib/api/objectGetAttributes.js Outdated Show resolved Hide resolved

francoisferrand reviewed Feb 17, 2026

View reviewed changes

lib/api/objectGetAttributes.js Outdated Show resolved Hide resolved

maeldonn force-pushed the feature/CLDSRV-817/get-object-attributes branch 2 times, most recently from caf1848 to ddde8bf Compare February 17, 2026 18:56

maeldonn added 2 commits February 17, 2026 20:04

Bump arsenal to 8.3.4

6e679a7

Issue: CLDSRV-817

Support the new API GetObjectAttributes

fd17bad

Issue: CLDSRV-817

maeldonn force-pushed the feature/CLDSRV-817/get-object-attributes branch from ddde8bf to fd17bad Compare February 17, 2026 19:04

maeldonn requested a review from francoisferrand February 17, 2026 19:17

scality deleted a comment from bert-e Feb 18, 2026

francoisferrand reviewed Feb 18, 2026

View reviewed changes

francoisferrand approved these changes Feb 18, 2026

View reviewed changes

bert-e merged commit fd17bad into development/9.3 Feb 18, 2026
71 of 73 checks passed

bert-e deleted the feature/CLDSRV-817/get-object-attributes branch February 18, 2026 16:26

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

Conversation

maeldonn commented Jan 26, 2026

Uh oh!

bert-e commented Jan 26, 2026

Hello maeldonn,

Uh oh!

bert-e commented Jan 26, 2026

Incorrect fix version

Uh oh!

codecov bot commented Jan 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❌ 1 Tests Failed:

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

DarkIsDude Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

maeldonn Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

bert-e commented Jan 28, 2026

Incorrect fix version

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

bert-e commented Jan 29, 2026

Incorrect fix version

Uh oh!

bert-e commented Jan 30, 2026

Incorrect fix version

Uh oh!

Uh oh!

Uh oh!

Uh oh!

bert-e commented Feb 12, 2026

Incorrect fix version

Uh oh!

bert-e commented Feb 12, 2026

Incorrect fix version

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

bert-e commented Feb 18, 2026

Waiting for approval

codecov bot commented Jan 26, 2026 •

edited

Loading