Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

Update JwtTokenExtractor.cs - #6685

Merged
tracyboehrer (tracyboehrer) merged 3 commits into
microsoft:mainfrom
xieofxie:xieofxie-patch
Aug 25, 2023
Merged

Update JwtTokenExtractor.cs#6685
tracyboehrer (tracyboehrer) merged 3 commits into
microsoft:mainfrom
xieofxie:xieofxie-patch

Conversation

@xieofxie

@xieofxie xieofxie commented Aug 22, 2023

Copy link
Copy Markdown
Contributor

Fixes #

#minor

Description

Use the used signing key's id instead of kid in header to check endorsement

Specific Changes

Testing

@ceciliaavila

Copy link
Copy Markdown
Collaborator

Hi xieofxie, could you explain the reason for this change?
Also, there's a failing unit test that will have to be updated.
Thanks.
image

@xieofxie

Copy link
Copy Markdown
Contributor Author

Cecilia Avila (@ceciliaavila) I have updated the test~

@tracyboehrer
tracyboehrer (tracyboehrer) merged commit 501ddca into microsoft:main Aug 25, 2023
This was referenced May 14, 2026
This was referenced May 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants