Skip to content

feat(security): refactor security configuration to permit health and version apis#164

Closed
DurgaPrasad-54 wants to merge 1 commit intoPSMRI:mainfrom
DurgaPrasad-54:release-3.6.2
Closed

feat(security): refactor security configuration to permit health and version apis#164
DurgaPrasad-54 wants to merge 1 commit intoPSMRI:mainfrom
DurgaPrasad-54:release-3.6.2

Conversation

@DurgaPrasad-54
Copy link
Copy Markdown
Contributor

@DurgaPrasad-54 DurgaPrasad-54 commented Mar 24, 2026

Summary by CodeRabbit

  • Refactor
    • Improved internal code organization for security configuration handling.

Note: This release includes internal code improvements with no user-facing changes.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 24, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ec6253d2-13d0-4082-b7ed-490f88bed022

📥 Commits

Reviewing files that changed from the base of the PR and between 121ce69 and 43a4974.

📒 Files selected for processing (1)
  • src/main/java/com/iemr/mmu/utils/mapper/SecurityConfig.java

📝 Walkthrough

Walkthrough

The security filter chain configuration in SecurityConfig was refactored to split the combined public request matchers into three separate individual matcher calls, maintaining identical security behavior across the unchanged CSRF, session, authentication, and exception handling configurations.

Changes

Cohort / File(s) Summary
Security Configuration Refactoring
src/main/java/com/iemr/mmu/utils/mapper/SecurityConfig.java
Decomposed consolidated requestMatchers() call into three separate matcher calls for "/user/**", "/health", and "/version" endpoints, each individually paired with .permitAll(). Functional behavior remains unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit hops through matchers three,
Once bundled tight, now split and free,
No logic changed, just rearranged,
Security stays fresh, nothing estranged! 🐰✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@sonarqubecloud
Copy link
Copy Markdown

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.

1 participant