Skip to content

Conversation

@keegan-caruso
Copy link
Contributor

Use IdentityModel 7.0.0-preview versions in Wilson7 branch

Description

Update the Wilson7 branch to use 7.0.0-preview identity model libraries

@keegan-caruso keegan-caruso requested review from a team and wtgodbe as code owners July 12, 2023 00:06
@ghost ghost added area-identity Includes: Identity and providers area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework community-contribution Indicates that the PR has been added by a community member labels Jul 12, 2023
@ghost
Copy link

ghost commented Jul 12, 2023

Thanks for your PR, @keegan-caruso. Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

Copy link
Member

@eerhardt eerhardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Merging and we will fix the CI failures in the Wilson7 branch.

@eerhardt eerhardt merged commit 4501b82 into dotnet:Wilson7 Jul 17, 2023
wtgodbe pushed a commit that referenced this pull request Jul 21, 2023
…49542)

* Option to use JsonWebTokenHandler in OpenIdConnectHandler

* fix sample

* split event tests

* update IdentityModel to 6.31.0

* Added JsonWebTokenHandler and TokenHandlers (#48857)

Co-authored-by: Brent Schmaltz <[email protected]>

* adjust for claims mapping
remove using System

* moved api's to unshipped

* Addressed PR comments

* Removed setter.

* Use 7.0.0-preview for identity model libraries

* fix bild break, useTokenHanlders default false.

* use var for identitymodel versions

* Move new apis to unshipped

* Increase key size to 256 bits or HMAC will fail.

* update version of identity model libraries (#49349)

Co-authored-by: Keegan Caruso <[email protected]>

* Update Wilson7 branch (#49491)

* SymmetricSecurityKey needs 32 bytes

* Update source-build-externals dependencies

---------

Co-authored-by: Keegan Caruso <[email protected]>

* Update Wilson7 branch (#49524)

* 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]>

* Changes from API review

* Comments from review

---------

Co-authored-by: Keegan Caruso <[email protected]>
Co-authored-by: Brent Schmaltz <[email protected]>
Co-authored-by: BrentSchmaltz <[email protected]>
Co-authored-by: Safia Abdalla <[email protected]>
eerhardt pushed a commit that referenced this pull request Jul 21, 2023
…49541)

* Option to use JsonWebTokenHandler in OpenIdConnectHandler

* fix sample

* split event tests

* update IdentityModel to 6.31.0

* Added JsonWebTokenHandler and TokenHandlers (#48857)

Co-authored-by: Brent Schmaltz <[email protected]>

* adjust for claims mapping
remove using System

* moved api's to unshipped

* Addressed PR comments

* Removed setter.

* Use 7.0.0-preview for identity model libraries

* fix bild break, useTokenHanlders default false.

* use var for identitymodel versions

* Move new apis to unshipped

* Increase key size to 256 bits or HMAC will fail.

* update version of identity model libraries (#49349)

Co-authored-by: Keegan Caruso <[email protected]>

* Update Wilson7 branch (#49491)

* SymmetricSecurityKey needs 32 bytes

* Update source-build-externals dependencies

---------

Co-authored-by: Keegan Caruso <[email protected]>

* Update Wilson7 branch (#49524)

* 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]>

* Changes from API review

* Comments from review

---------

Co-authored-by: Keegan Caruso <[email protected]>
Co-authored-by: Brent Schmaltz <[email protected]>
Co-authored-by: BrentSchmaltz <[email protected]>
Co-authored-by: Safia Abdalla <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-identity Includes: Identity and providers area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants