[OB4] Fix spotbugs issue in debug log

[imesh94]

Purpose

Fix spotbugs issue in debug log

Summary by CodeRabbit

• Chores
  • Applied enhanced security annotations to identity service components
  • Removed debug logging from regulatory application verification processes

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Warning

Rate limit exceeded

@imesh94 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 4 minutes and 45 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 932a07b and e18ce2b.

📒 Files selected for processing (7)

• financial-services-accelerator/components/org.wso2.financial.services.accelerator.identity.extensions/src/main/java/org/wso2/financial/services/accelerator/identity/extensions/auth/extensions/request/validator/FSRequestObjectValidationExtension.java (2 hunks)
• financial-services-accelerator/components/org.wso2.financial.services.accelerator.identity.extensions/src/main/java/org/wso2/financial/services/accelerator/identity/extensions/auth/extensions/response/handler/FSCodeResponseTypeHandlerExtension.java (2 hunks)
• financial-services-accelerator/components/org.wso2.financial.services.accelerator.identity.extensions/src/main/java/org/wso2/financial/services/accelerator/identity/extensions/auth/extensions/response/handler/FSHybridResponseTypeHandlerExtension.java (2 hunks)
• financial-services-accelerator/components/org.wso2.financial.services.accelerator.identity.extensions/src/main/java/org/wso2/financial/services/accelerator/identity/extensions/claims/FSDefaultOIDCClaimsCallbackHandler.java (2 hunks)
• financial-services-accelerator/components/org.wso2.financial.services.accelerator.identity.extensions/src/main/java/org/wso2/financial/services/accelerator/identity/extensions/grant/type/handlers/FSAuthorizationCodeGrantHandler.java (2 hunks)
• financial-services-accelerator/components/org.wso2.financial.services.accelerator.identity.extensions/src/main/java/org/wso2/financial/services/accelerator/identity/extensions/grant/type/handlers/FSRefreshGrantHandler.java (2 hunks)
• financial-services-accelerator/components/org.wso2.financial.services.accelerator.identity.extensions/src/main/java/org/wso2/financial/services/accelerator/identity/extensions/util/IdentityCommonUtils.java (1 hunks)

Walkthrough

A utility class was updated to add a FindBugs suppression annotation for CRLF injection in logging and to remove a debug log statement from the isRegulatoryApp method, simplifying the code without altering functionality.

Changes

Cohort / File(s)	Summary
Logging and annotation cleanup financial-services-accelerator/components/org.wso2.financial.services.accelerator.identity.extensions/src/main/java/org/wso2/financial/services/accelerator/identity/extensions/util/IdentityCommonUtils.java	Added class-level FindBugs CRLF_INJECTION_LOGS suppression annotation with justification for sanitized external inputs; removed debug log in isRegulatoryApp method when regulatory property is null.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A log was removed, neat and clean,
Suppression added, a protective screen,
One file touched with a gentle hand,
Code now flows as was always planned! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is minimal and lacks required sections such as issue link, development checklist, secure development checklist, and testing checklist as specified in the template.	Add required sections from the template including issue link, development and secure development checklists, and testing details to meet repository standards.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the main change: fixing a spotbugs issue related to debug logging.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b792935 and ae97f7d.

📒 Files selected for processing (1)

• financial-services-accelerator/components/org.wso2.financial.services.accelerator.identity.extensions/src/main/resources/findbugs-exclude.xml (1 hunks)