Skip to content
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

Remove Outdated Comment on TokenDecryptionCertificates #1511

Merged
merged 1 commit into from
Nov 3, 2021

Conversation

michaelwstark
Copy link
Contributor

There is a comment on the public class MicrosoftIdentityOptions.cs which indicates only the first certificate in the set is considered. Reading through the code as well as looking over historical PRs, it appears this limitation was resolved.

#905

Correcting comment to save others from the same investigation.

@jennyf19
Copy link
Collaborator

jennyf19 commented Nov 3, 2021

Great catch. Thank you @michaelwstark

@jennyf19 jennyf19 merged commit c95d880 into AzureAD:master Nov 3, 2021
@jennyf19
Copy link
Collaborator

jennyf19 commented Nov 5, 2021

@michaelwstark part of 1.20 release

@jennyf19 jennyf19 added this to the 1.20 milestone Nov 5, 2021
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