Skip to content

CME-968: feat: enhance JurisdictionMatcher to support cross-jurisdict…#2849

Open
AntonyLeons wants to merge 3 commits into
masterfrom
CME-968-fail-open-design
Open

CME-968: feat: enhance JurisdictionMatcher to support cross-jurisdict…#2849
AntonyLeons wants to merge 3 commits into
masterfrom
CME-968-fail-open-design

Conversation

@AntonyLeons

Copy link
Copy Markdown
Contributor

…ional roles and update tests

Before creating a pull request make sure that:

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • the keep-helm label has been added, if the helm release should be persisted after a successful build

Please remove this line and everything above and fill the following sections:

JIRA link (if applicable)

CME-968

Change description

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[ ] No


@Autowired
public JurisdictionMatcher(ApplicationParams applicationParams) {
this.crossJurisdictionalRoles = applicationParams.getCcdAccessControlCrossJurisdictionRoles();

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

roles already in applicationparams here, assuming all roles are included and therefore safe to deploy

@hmcts-jenkins-a-to-c hmcts-jenkins-a-to-c Bot requested a deployment to preview April 21, 2026 11:35 Abandoned
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants