Skip to content

Accept 2 check_claims configs and try them both - #52

Draft
JonaBenja wants to merge 1 commit into
mainfrom
jona/fix_keycloak_aud
Draft

Accept 2 check_claims configs and try them both#52
JonaBenja wants to merge 1 commit into
mainfrom
jona/fix_keycloak_aud

Conversation

@JonaBenja

@JonaBenja JonaBenja commented Aug 25, 2026

Copy link
Copy Markdown

Gelinkt aan dit ticket:
https://dev.azure.com/CloudCompetenceCenter/Data%20Diensten/_workitems/edit/244680

aud en iss claims worden nu bij elk token gecheckt, niet alleen bij Entra tokens. Dit heeft als gevolg dat de huidige tests aangepast moeten worden, want die gaan hier nu niet vanuit. Ik heb daar een beginnetje aan gemaakt, maar dit moet nog goed nagelopen worden.

  • De test coverage is te laag (denk door de nieuwe code die ik heb toegevoegd), daardoor falen de tests nu

@JonaBenja
JonaBenja force-pushed the jona/fix_keycloak_aud branch 3 times, most recently from 7f7e798 to 72264e0 Compare August 25, 2026 14:19
@JonaBenja
JonaBenja force-pushed the jona/fix_keycloak_aud branch from 72264e0 to 302e753 Compare August 25, 2026 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant