AMP-204 - Change webhook to callback pattern for notification api call to client by srivanimuddineni · Pull Request #29 · hmcts/service-cp-crime-hearing-case-event-subscription

srivanimuddineni · 2026-01-29T14:05:40Z

Updated the API specification for subscription dependencies and made corresponding changes to related areas for callback Url
Added a test-only advice to prevent conflicts with the production globalExceptionHandler - to handle validation and not-found exceptions specific to integration tests.
Added standard Jackson ObjectMapper for few ITs
Refactored the package structure for both unit and integration test cases.
Added URL sanitization while logging - don't think this is required as it will be taken care on APIM policy. However added to get rid of PMD issue fyi

src/main/java/uk/gov/hmcts/cp/subscription/controllers/SubscriptionController.java

Fixed Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

src/main/java/uk/gov/hmcts/cp/subscription/utils/SubscriptionHelper.java

src/main/java/uk/gov/hmcts/cp/subscription/controllers/advice/GlobalExceptionHandler.java

src/main/java/uk/gov/hmcts/cp/subscription/mappers/SubscriptionMapper.java

...a/uk/gov/hmcts/cp/subscription/integration/clients/DocumentServiceClientIntegrationTest.java

...st/java/uk/gov/hmcts/cp/subscription/integration/services/WebhookServiceIntegrationTest.java

src/main/java/uk/gov/hmcts/cp/subscription/controllers/SubscriptionController.java

srivanimuddineni added 2 commits January 29, 2026 13:59

AMP-204 changes

94fd9c8

changes

e623bfb

srivanimuddineni requested a review from a team as a code owner January 29, 2026 14:05

github-advanced-security bot found potential problems Jan 29, 2026

View reviewed changes

src/main/java/uk/gov/hmcts/cp/subscription/controllers/SubscriptionController.java Fixed Show fixed Hide fixed

srivanimuddineni and others added 4 commits January 29, 2026 14:49

pmd issues fix

84efa97

Added sanitized url at logs

d0c3e49

Potential fix for code scanning alert no. 1: Log Injection

b987b1f

Fixed Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

Added url sanitization

95ce2b5

srivanimuddineni requested review from coling01 and samirgarg January 29, 2026 15:17