Skip to content

Allow metadata url keyrollover #197

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

Merged
merged 1 commit into from
Jun 10, 2025

Conversation

pablothedude
Copy link
Contributor

@pablothedude pablothedude commented Jun 4, 2025

This is needed in order to integrate an identity provider with the help
of a metadata url when doing a request. So an automatic key rollover
can take place and the IdP config is cached from the metadata url.

The update of the config is done in the callout itself and is not a
separate flow. This was done to prevent downtime because only one
certificate is in the metadata endpoint of the Microsoft identity
providers.

#194

@pablothedude pablothedude force-pushed the feature/allow-metadata-url-keyrollover branch 23 times, most recently from f378931 to 63bc617 Compare June 6, 2025 11:17
@pablothedude pablothedude requested a review from parijke June 6, 2025 11:22
This is needed in order to integrate an identity provider with the help
of a metadata url when doing a request. So an automatic key rollover
can take place and the IdP config is cached from the metadata url.

The update of the config is done in the callout itself and is not a
separate flow. This was done to prevent downtime because only one
certificate is in the metadata endpoint of the Microsoft identity
providers.

#194
@pablothedude pablothedude force-pushed the feature/allow-metadata-url-keyrollover branch from 63bc617 to 0d23474 Compare June 10, 2025 07:51
@pablothedude pablothedude merged commit 79065bc into main Jun 10, 2025
2 checks passed
@pablothedude pablothedude deleted the feature/allow-metadata-url-keyrollover branch June 10, 2025 08:39
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