Skip to content

Cherry-pick Cache context switches (#2724)#2775

Open
westin-m wants to merge 10 commits intodev6xfrom
westin/switch-cache
Open

Cherry-pick Cache context switches (#2724)#2775
westin-m wants to merge 10 commits intodev6xfrom
westin/switch-cache

Conversation

@westin-m
Copy link
Copy Markdown
Contributor

@westin-m westin-m commented Aug 9, 2024

Include changes to app context switch in 6x.

* Cache context switches.

* Refactor switch reset in tests.

* Rename.

* Refactor to reset switches in one method.

* Fix tests.

* Fix tests.
@westin-m westin-m marked this pull request as ready for review August 9, 2024 19:05
@keegan-caruso keegan-caruso requested a review from pmaytak August 9, 2024 19:15
Comment thread src/Microsoft.IdentityModel.JsonWebTokens/JwtTokenUtilities.cs Outdated
Comment thread src/Microsoft.IdentityModel.Tokens/Json/JsonSerializerPrimitives.cs Outdated
Comment thread src/Microsoft.IdentityModel.Validators/AadTokenValidationParametersExtension.cs Outdated
westin-m and others added 3 commits August 9, 2024 12:30
Co-authored-by: Keegan Caruso <Keegan.Caruso@microsoft.com>
@westin-m westin-m requested a review from keegan-caruso August 9, 2024 19:40
@westin-m westin-m linked an issue Aug 9, 2024 that may be closed by this pull request
Comment thread src/Microsoft.IdentityModel.JsonWebTokens/JwtTokenUtilities.cs Outdated
@westin-m westin-m requested a review from keegan-caruso August 9, 2024 21:53
Comment thread src/Microsoft.IdentityModel.Tokens/AppContextSwitches.cs Outdated
Copy link
Copy Markdown
Contributor

@keegan-caruso keegan-caruso left a comment

Choose a reason for hiding this comment

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

Please do a check over AppContextSwitches and make sure we only have what we need.

Comment thread src/Microsoft.IdentityModel.Tokens/AppContextSwitches.cs Outdated
Comment thread src/Microsoft.IdentityModel.Tokens/ClaimsIdentityFactory.cs
Comment thread src/Microsoft.IdentityModel.Validators/AadTokenValidationParametersExtension.cs Outdated
Comment thread src/Microsoft.IdentityModel.Tokens/AppContextSwitches.cs
@westin-m westin-m requested a review from pmaytak January 9, 2025 19:39
@keegan-caruso keegan-caruso self-requested a review January 30, 2025 19:31
@John0King
Copy link
Copy Markdown

what happend to this pr ?

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.

[Feature Request] Cherry-pick app switch caching into 6x

4 participants