Skip to content

INTEGRATION [PR#394 > development/1] Bugfix/s3 utils 239 fix count items bucket logging#398

Merged
bert-e merged 6 commits into
development/1from
w/1/bugfix/S3UTILS-239-fix-count-items-bucket-logging
Jun 10, 2026
Merged

INTEGRATION [PR#394 > development/1] Bugfix/s3 utils 239 fix count items bucket logging#398
bert-e merged 6 commits into
development/1from
w/1/bugfix/S3UTILS-239-fix-count-items-bucket-logging

Conversation

@bert-e

@bert-e bert-e commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

This pull request has been created automatically.
It is linked to its parent pull request #394.

Do not edit this pull request directly.
If you need to amend/cancel the changeset on branch
w/1/bugfix/S3UTILS-239-fix-count-items-bucket-logging, please follow this
procedure:

 git fetch
 git checkout w/1/bugfix/S3UTILS-239-fix-count-items-bucket-logging
 # <amend or cancel the changeset by _adding_ new commits>
 git push origin w/1/bugfix/S3UTILS-239-fix-count-items-bucket-logging

Please always comment pull request #394 instead of this one.

@claude

claude Bot commented Jun 10, 2026

Copy link
Copy Markdown

LGTM

The fix correctly extracts the bucket info data before nullifying unsupported serialized fields, which also fixes the pre-existing bug where _websiteConfiguration was checked on the wrapper object instead of the actual bucket info data when wrapped in { bucketInfo: {...} }. The _bucketLoggingStatus nullification follows the same established pattern. Test coverage is adequate.

Review by Claude Code

@bert-e bert-e merged commit 482a298 into development/1 Jun 10, 2026
12 checks passed
@bert-e bert-e deleted the w/1/bugfix/S3UTILS-239-fix-count-items-bucket-logging branch June 10, 2026 16:09
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.

2 participants