Skip to content

W/8.8/bugfix/cldsrv 674#5884

Merged
bert-e merged 6 commits intodevelopment/8.8from
w/8.8/bugfix/CLDSRV-674
Jul 23, 2025
Merged

W/8.8/bugfix/cldsrv 674#5884
bert-e merged 6 commits intodevelopment/8.8from
w/8.8/bugfix/CLDSRV-674

Conversation

@leif-scality
Copy link
Copy Markdown
Contributor

Handle cross account cases when using bucket policies

https://scality.atlassian.net/browse/S3C-9896

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 23, 2025

Codecov Report

Attention: Patch coverage is 85.86957% with 13 lines in your changes missing coverage. Please review.

Project coverage is 65.27%. Comparing base (766ecba) to head (4579513).
Report is 6 commits behind head on development/8.8.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
lib/api/apiUtils/authorization/permissionChecks.js 92.30% 6 Missing ⚠️
lib/metadata/metadataUtils.js 53.84% 6 Missing ⚠️
lib/api/api.js 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

Files with missing lines Coverage Δ
lib/api/objectGet.js 49.11% <ø> (ø)
lib/api/api.js 80.42% <0.00%> (-0.43%) ⬇️
lib/api/apiUtils/authorization/permissionChecks.js 90.90% <92.30%> (+3.22%) ⬆️
lib/metadata/metadataUtils.js 76.19% <53.84%> (-2.94%) ⬇️
@@                 Coverage Diff                 @@
##           development/8.8    #5884      +/-   ##
===================================================
+ Coverage            65.11%   65.27%   +0.15%     
===================================================
  Files                  188      188              
  Lines                11961    12024      +63     
===================================================
+ Hits                  7789     7849      +60     
- Misses                4172     4175       +3     
Flag Coverage Δ
unit 65.27% <85.86%> (+0.15%) ⬆️

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.

@bert-e bert-e closed this Jul 23, 2025
@bert-e bert-e merged commit 4579513 into development/8.8 Jul 23, 2025
45 of 46 checks passed
@bert-e bert-e deleted the w/8.8/bugfix/CLDSRV-674 branch July 23, 2025 14:11
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