Skip to content

LPT-2121#127

Open
ynegi wants to merge 11 commits intoteam/cct-1981from
LPT-2121
Open

LPT-2121#127
ynegi wants to merge 11 commits intoteam/cct-1981from
LPT-2121

Conversation

@ynegi
Copy link
Contributor

@ynegi ynegi commented Mar 18, 2026

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)

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

JIRA link (if applicable)

Change description

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

[ ] Yes
[ ] No

@ynegi ynegi requested a review from arcadiushmcts March 18, 2026 14:28
/** POST /schedule-job: system user only. Schedules cleanup at given cron and runs once immediately. */
@RestController
@Slf4j
public class CleanupController {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

followed the spec now

.parseSignedClaims(token)
.getPayload();
final String userName = claims.getSubject();
final String subject = claims.getSubject();
Copy link
Collaborator

Choose a reason for hiding this comment

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

Note that for system-user check, you can add a rule to https://github.com/hmcts/cp-court-list-publishing-service/blob/team/cct-1981/src/main/resources/acl/court-list-publishing-rules.drl and no JWT changes is required

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added the system users in drl file

@Setter
@Component
@Slf4j
public class CleanupJob extends QuartzJobBean {
Copy link
Collaborator

Choose a reason for hiding this comment

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

No quartz required.... just an endpoint is needed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed Quartz

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.

2 participants