Skip to content

A quick update to change the SP used for the Test OIDC links from /test/oidc/login.#10822

Merged
solipet merged 1 commit intomainfrom
update-oidc-test-controller
Jun 14, 2024
Merged

A quick update to change the SP used for the Test OIDC links from /test/oidc/login.#10822
solipet merged 1 commit intomainfrom
update-oidc-test-controller

Conversation

@solipet
Copy link
Copy Markdown
Contributor

@solipet solipet commented Jun 14, 2024

A quick update to change the SP used for the Test OIDC links from /test/oidc/login.

I found the existing SP failed with errors for me. This changes makes the links work again.

Note: the test routes are disabled in production envs (i.e., deployed envs)

📜 Testing Plan

Run your local server and go to /test/oidc/login and try the links.

/test/oidc/login.

I found the existing SP failed with errors for me. This changes makes
the links work again.

Note: the test routes are disabled in production envs (i.e., deployed
envs)

[skip changelog]
@solipet solipet requested review from amirbey, jmax-gsa and theabrad June 14, 2024 20:19
@solipet solipet merged commit ec59a21 into main Jun 14, 2024
@solipet solipet deleted the update-oidc-test-controller branch June 14, 2024 22:31
brandemix pushed a commit to brandemix/18F-identity-idp that referenced this pull request Jun 17, 2024
…F#10822)

/test/oidc/login.

I found the existing SP failed with errors for me. This changes makes
the links work again.

Note: the test routes are disabled in production envs (i.e., deployed
envs)

[skip changelog]
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