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

Update Wilson7 branch #49491

Merged
merged 2 commits into from
Jul 18, 2023
Merged

Update Wilson7 branch #49491

merged 2 commits into from
Jul 18, 2023

Conversation

captainsafia
Copy link
Member

No description provided.

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-signalr Includes: SignalR clients and servers label Jul 18, 2023
@captainsafia captainsafia added area-auth Includes: Authn, Authz, OAuth, OIDC, Bearer and removed area-signalr Includes: SignalR clients and servers labels Jul 18, 2023
@captainsafia captainsafia changed the title Keegan caruso/fix wilson7 tests Update Wilson7 branch Jul 18, 2023
@captainsafia captainsafia marked this pull request as ready for review July 18, 2023 15:36
Copy link
Contributor

@jmprieur jmprieur left a comment

Choose a reason for hiding this comment

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

LGTM
Thanks @captainsafia

Copy link
Contributor

@jennyf19 jennyf19 left a comment

Choose a reason for hiding this comment

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

:shipit:

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

@captainsafia captainsafia merged commit 583728e into Wilson7 Jul 18, 2023
@captainsafia captainsafia deleted the keegan-caruso/fix-wilson7-tests branch July 18, 2023 15:43
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-auth Includes: Authn, Authz, OAuth, OIDC, Bearer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants