Skip to content

CCD-7109 JWT Issuer Validation Disabled in Multiple Services#763

Open
patelila wants to merge 8 commits intomasterfrom
CCD-7109_JWT_Issuer_Validation_Disabled
Open

CCD-7109 JWT Issuer Validation Disabled in Multiple Services#763
patelila wants to merge 8 commits intomasterfrom
CCD-7109_JWT_Issuer_Validation_Disabled

Conversation

@patelila
Copy link
Copy Markdown
Contributor

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)

Change description

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

[ ] Yes
[ ] No

IDAM_API_URL: https://idam-api.{{ .Values.global.environment }}.platform.hmcts.net
IDAM_OIDC_URL: https://idam-web-public.{{ .Values.global.environment }}.platform.hmcts.net
OIDC_ISSUER: https://forgerock-am.service.core-compute-idam-{{ .Values.global.environment }}.internal:8443/openam/oauth2/hmcts
OIDC_ISSUER: https://forgerock-am.service.core-compute-idam-aat2.internal:8443/openam/oauth2/realms/root/realms/hmcts
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

you are replacing {{ .Values.global.environment }} with aat2 here. Is this correct for all other environments given it is already being overridden in preview.template.yaml ?

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