Skip to content

fix(oci): Mutelist support#10566

Merged
HugoPBrito merged 4 commits into
prowler-cloud:masterfrom
rchotacode:fix/fix-oci-mutelist
Apr 7, 2026
Merged

fix(oci): Mutelist support#10566
HugoPBrito merged 4 commits into
prowler-cloud:masterfrom
rchotacode:fix/fix-oci-mutelist

Conversation

@rchotacode
Copy link
Copy Markdown
Contributor

Context

Please include relevant motivation and context for this PR.

Mutelist files are currently unsupported for OCI. This change makes them functional
Fix #10565

Description

Updates oraclecloud_mutelist_example.yaml to reflect current correct schema. Adds in required mutelist argument tenancy_id

Steps to review

Running any mutelist schema

Checklist

Community Checklist
  • This feature/issue is listed in here or roadmap.prowler.com
  • Is it assigned to me, if not, request it via the issue/feature in here or Prowler Community Slack

SDK/CLI

  • Are there new checks included in this PR? Yes / No
    • If so, do we need to update permissions for the provider? Please review this carefully.

UI

  • All issue/task requirements work as expected on the UI
  • Screenshots/Video of the functionality flow (if applicable) - Mobile (X < 640px)
  • Screenshots/Video of the functionality flow (if applicable) - Table (640px > X < 1024px)
  • Screenshots/Video of the functionality flow (if applicable) - Desktop (X > 1024px)
  • Ensure new entries are added to CHANGELOG.md, if applicable.

API

  • All issue/task requirements work as expected on the API
  • Endpoint response output (if applicable)
  • EXPLAIN ANALYZE output for new/modified queries or indexes (if applicable)
  • Performance test results (if applicable)
  • Any other relevant evidence of the implementation (if applicable)
  • Verify if API specs need to be regenerated.
  • Check if version updates are required (e.g., specs, Poetry, etc.).
  • Ensure new entries are added to CHANGELOG.md, if applicable.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@rchotacode rchotacode requested a review from a team April 2, 2026 17:16
@rchotacode rchotacode requested a review from a team as a code owner April 2, 2026 17:16
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 2, 2026

Conflict Markers Resolved

All conflict markers have been successfully resolved in this pull request.

@github-actions github-actions Bot added the community Opened by the Community label Apr 2, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 2, 2026

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 64.08%. Comparing base (509ec74) to head (2b4b212).
⚠️ Report is 16 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (509ec74) and HEAD (2b4b212). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (509ec74) HEAD (2b4b212)
api 1 0
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #10566       +/-   ##
===========================================
- Coverage   93.52%   64.08%   -29.44%     
===========================================
  Files         225      185       -40     
  Lines       31579     9796    -21783     
===========================================
- Hits        29534     6278    -23256     
- Misses       2045     3518     +1473     
Flag Coverage Δ
api ?
prowler-py3.10-config 64.08% <66.66%> (?)
prowler-py3.10-lib 63.89% <66.66%> (?)
prowler-py3.10-oraclecloud 59.88% <0.00%> (?)
prowler-py3.11-config 64.08% <66.66%> (?)
prowler-py3.11-lib 63.89% <66.66%> (?)
prowler-py3.11-oraclecloud 59.88% <0.00%> (?)
prowler-py3.12-config 64.08% <66.66%> (?)
prowler-py3.12-lib 63.89% <66.66%> (?)
prowler-py3.12-oraclecloud 59.88% <0.00%> (?)

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

Components Coverage Δ
prowler 64.08% <66.66%> (∅)
api ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rchotacode rchotacode changed the title fix(oci): minimal mutelist fix fix(oci): Mutelist support Apr 2, 2026
@rchotacode
Copy link
Copy Markdown
Contributor Author

Before change:
image
image
After Change:
image

@HugoPBrito HugoPBrito self-assigned this Apr 6, 2026
@HugoPBrito HugoPBrito added the provider/oci Issues/PRs related with the OCI provider label Apr 6, 2026
- Move changelog entry from Changed to Fixed (it's a bugfix)
- Add test_execute_oraclecloud_mutelist_passes_tenancy_id
- Fix remaining 'Tenancies' reference in yaml comment

Ref: prowler-cloud#10565
oci.config.validate_config requires the 'user' field which is not
present in session token configs created by 'oci session authenticate'.
Moving the validation after token detection ensures it only runs for
API key auth, where 'user' is always present.
@HugoPBrito HugoPBrito merged commit 5e1e4bd into prowler-cloud:master Apr 7, 2026
33 of 35 checks passed
@HugoPBrito
Copy link
Copy Markdown
Member

Thank you for your contribution ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Opened by the Community provider/oci Issues/PRs related with the OCI provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[OCI] Mutelist yaml failing on is_finding_muted

2 participants