Skip to content

fix(matcher): update securityContext to run as root#1133

Merged
guimoreirar merged 1 commit intomainfrom
hotfix/matcher-security-context
Mar 18, 2026
Merged

fix(matcher): update securityContext to run as root#1133
guimoreirar merged 1 commit intomainfrom
hotfix/matcher-security-context

Conversation

@guimoreirar
Copy link
Member

@guimoreirar guimoreirar commented Mar 18, 2026

Midaz Pull Request Checklist

Pull Request Type

  • Midaz
  • Plugin Access Manager
  • Plugin CRM
  • Reporter
  • Plugin Fees
  • Plugin BR PIX Direct JD
  • Plugin BR PIX Indirect BTG
  • Otel Collector
  • Pipeline
  • Documentation
  • Fetcher
  • Matcher
  • Flowker
  • Underwriter

Checklist

Please check each item after it's completed.

  • I have tested these changes locally.
  • I have updated the documentation accordingly.
  • I have added necessary comments to the code, especially in complex areas.
  • I have ensured that my changes adhere to the project's coding standards.
  • I have checked for any potential security issues.
  • I have ensured that all tests pass.
  • I have updated the version appropriately (if applicable).
  • I have confirmed this code is ready for review.

Additional Notes

Obs: Please, always remember to target your PR to develop branch instead of main.

@guimoreirar guimoreirar self-assigned this Mar 18, 2026
@guimoreirar guimoreirar requested a review from a team as a code owner March 18, 2026 17:37
@guimoreirar guimoreirar merged commit bfbb6cb into main Mar 18, 2026
4 of 5 checks passed
@guimoreirar guimoreirar deleted the hotfix/matcher-security-context branch March 18, 2026 17:37
@coderabbitai
Copy link

coderabbitai bot commented Mar 18, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 541e40c6-5346-44da-9111-a0983e1b1120

📥 Commits

Reviewing files that changed from the base of the PR and between b42f99f and 8de4a13.

📒 Files selected for processing (1)
  • charts/matcher/values.yaml

Walkthrough

This change modifies the security context configuration for the matcher service in the Helm chart values. The container now runs as the root user (runAsUser and runAsGroup set to 0) instead of non-root, and the root filesystem is configured as writable instead of read-only. The runAsNonRoot flag is set to false to align with these changes.

📝 Coding Plan
  • Generate coding plan for human review comments

Comment @coderabbitai help to get the list of available commands and usage tips.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant