Skip to content

Add rollover metadata containing all available keys#1899

Open
tvdijen wants to merge 2 commits intoOpenConext:mainfrom
tvdijen:feature/rollover-metadata
Open

Add rollover metadata containing all available keys#1899
tvdijen wants to merge 2 commits intoOpenConext:mainfrom
tvdijen:feature/rollover-metadata

Conversation

@tvdijen
Copy link
Contributor

@tvdijen tvdijen commented Dec 16, 2025

We sign our AuthnRequests to our IdPs, so we need to have metadata containing both the current and the rollover-certificate.
With this patch, the default metadata (keyslug=default or no keyslug) will contain all the configured keys.

@tvdijen tvdijen force-pushed the feature/rollover-metadata branch from 07e9c6c to 38b72e8 Compare December 16, 2025 11:41
@baszoetekouw baszoetekouw requested a review from johanib January 7, 2026 07:56
@cdbesten
Copy link

Dit lijkt me zeer gewenst. Wij moeten volgend jaar op productie ook rollover'en en alles wat dit proces soepeler kan laten verlopen is mooi meegenomen.

@johanib johanib moved this from New to Backlog in PHP development Feb 23, 2026
@johanib
Copy link
Contributor

johanib commented Mar 18, 2026

@kayjoosten Can you:

  • Add .feature test
  • Check test coverage (unit needed/possible)
  • Review implementation
  • Assume current implementation is correct

@kayjoosten kayjoosten self-assigned this Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

4 participants