Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DTSAM-572 Audit use of case-allocator rules for CCD case-roles #2365

Merged
merged 27 commits into from
Nov 19, 2024

Conversation

mikebrownccd
Copy link
Contributor

JIRA link (if applicable)

https://tools.hmcts.net/jira/browse/DTSAM-572

Change description

DTSAM-572 Audit use of case-allocator rules for CCD case-roles

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

[ ] Yes
[x] No

@mikebrownccd mikebrownccd changed the base branch from master to DTSAM-571_Aduit_byPassOrgDroolRule November 12, 2024 16:35
mattnayler
mattnayler previously approved these changes Nov 13, 2024
@mikebrownccd mikebrownccd removed the enable_keep_helm Do not uninstall helm release after pipeline completion label Nov 19, 2024
Base automatically changed from DTSAM-571_Aduit_byPassOrgDroolRule to master November 19, 2024 11:22
@mikebrownccd mikebrownccd dismissed stale reviews from mattnayler and abul-syed November 19, 2024 11:22

The base branch was changed.

Copy link
Contributor

@mattnayler mattnayler left a comment

Choose a reason for hiding this comment

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

RE-approved after base branch change.

Copy link
Contributor

@abul-syed abul-syed left a comment

Choose a reason for hiding this comment

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

lgtm

@mikebrownccd mikebrownccd merged commit 7a38104 into master Nov 19, 2024
5 checks passed
@mikebrownccd mikebrownccd deleted the DTSAM-572_Audit_Case_Allocator_Approve branch November 19, 2024 11:46
mikebrownccd added a commit that referenced this pull request Nov 21, 2024
* CME-121 Database monitoring and alerting on AM repositories - Unsync ITHC (#2379)

* DTSAM-611 Fix RAS FTA issue with XUI S2S token (#2385)

[DTSAM-611](https://tools.hmcts.net/jira/browse/DTSAM-611)

Refactor S2S token secrets and environment variables used in FTAs.

* DTSAM-571 Audit byPassOrgDroolRule when receiving ORG roles from ORM (#2364)

* DTSAM-571 Audit byPassOrgDroolRule when receiving ORG roles from ORM

* process included in log

* () added to log properties

* do not log if client id = am_org_role_mapping_service

* not ORM log filter fixed by using .equals()

* logging made null safe

* FTAs with ref S-*_Org_Role_Creation added ORM request header to remove from the KQL report

* ServiceAuthorization header moved to Test_Data_Base file

* ServiceAuthorization header added to requests in common dir

* DTSAM-572 Audit use of case-allocator rules for CCD case-roles (#2365)

* DTSAM-571 Audit byPassOrgDroolRule when receiving ORG roles from ORM

* process included in log

* () added to log properties

* do not log if client id = am_org_role_mapping_service

* not ORM log filter fixed by using .equals()

* DTSAM-572 Audit use of case-allocator rules for CCD case-roles - conditional logInfoMsg() added

* temp removal of conditional for report dev in aat

* temp removal of conditional for report dev in aat

* space added to log

* logging made null safe

* RoleCategory filter uncommented

* logging made null safe

* FTAs with ref S-*_Org_Role_Creation added ORM request header to remove from the KQL report

* ServiceAuthorization header moved to Test_Data_Base file

* ServiceAuthorization header added to requests in common dir

* logged rule corrected to case_allocator_approve_delete_case_role_all_wa_services

* RoleCategory comparisons made null safe

* Renovate Combined Pull Request (#2382)

* Update dependency org.pitest:pitest to v1.17.1

* Update versions.tomcat to v9.0.97

* DTSAM-610 Renovate review W/C 2024-11-18 - pitest reverted 1.17.1 -> 1.17.0

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: github-actions <github-actions@github.com>
Co-authored-by: Mike Brown <michael.brown@hmcts.net>
Co-authored-by: mikebrownccd <104495891+mikebrownccd@users.noreply.github.com>

---------

Co-authored-by: mikebrownccd <104495891+mikebrownccd@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: github-actions <github-actions@github.com>
Co-authored-by: Mike Brown <michael.brown@hmcts.net>
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.

3 participants