-
Notifications
You must be signed in to change notification settings - Fork 10.1k
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
Update JwtBearer, WsFed, and OIDC handlers to use identity model 7 #49541
Merged
eerhardt
merged 21 commits into
dotnet:main
from
keegan-caruso:keegan-caruso/wilson7-consolidated
Jul 21, 2023
Merged
Update JwtBearer, WsFed, and OIDC handlers to use identity model 7 #49541
eerhardt
merged 21 commits into
dotnet:main
from
keegan-caruso:keegan-caruso/wilson7-consolidated
Jul 21, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: Brent Schmaltz <[email protected]>
remove using System
Co-authored-by: Keegan Caruso <[email protected]>
* SymmetricSecurityKey needs 32 bytes * Update source-build-externals dependencies --------- Co-authored-by: Keegan Caruso <[email protected]>
* Update Wilson7 branch Default to using new handlers Changes from API review * Handle obsolete members * Handle obsolete members in tests * Add aka.ms link --------- Co-authored-by: Keegan Caruso <[email protected]>
…/wilson7-consolidated
keegan-caruso
requested review from
BrennanConroy,
halter73,
Tratcher,
a team and
wtgodbe
as code owners
July 20, 2023 15:57
ghost
added
the
community-contribution
Indicates that the PR has been added by a community member
label
Jul 20, 2023
Thanks for your PR, @keegan-caruso. Someone from the team will get assigned to your PR shortly and we'll get it reviewed. |
dotnet-issue-labeler
bot
added
the
area-auth
Includes: Authn, Authz, OAuth, OIDC, Bearer
label
Jul 20, 2023
eerhardt
approved these changes
Jul 21, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving as this is what was committed into the release/8.0-preview7 branch.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area-auth
Includes: Authn, Authz, OAuth, OIDC, Bearer
community-contribution
Indicates that the PR has been added by a community member
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update JwtBearer, WsFed, and OIDC handlers to use identity model 7
Description
IdentityModel has updated the token validation to be more performant, provide an async model and provide a Last-Known-Good model for preserving metadata that was last known to validate a token for a specific identity provider
Proposing that we consolidate #48966 and #49333 into this one