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

Bau/fix jwks http file #5741

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Bau/fix jwks http file #5741

wants to merge 2 commits into from

Conversation

alhcomer
Copy link
Contributor

What

jwks-host was wrong in http-client.env.json for the dev environment. This has been fixed

@alhcomer alhcomer requested review from a team as code owners January 13, 2025 16:54
Copy link

Java Tests Skipped

No Java files were changed in this pull request. Java tests will be skipped1.

Any Java files that are changed in a subsequent commit will trigger the Java tests.

Footnotes

  1. These tests will still show as passing in the PR status check, but will not actually have run.

@@ -1,7 +1,7 @@
{
"dev": {
"env": "dev",
"jwks-host": "941urm75id.execute-api.eu-west-2.amazonaws.com"
"jwks-host": "auth.staging.account.gov.uk"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"jwks-host": "auth.staging.account.gov.uk"
"jwks-host": "auth.dev.account.gov.uk"

@alhcomer alhcomer force-pushed the BAU/fix-jwks-http-file branch from f701fe7 to d5b2951 Compare January 13, 2025 16:57
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