Skip to content

Replace identitymodel#409

Merged
happymancoder merged 3 commits into
masterfrom
replace-identitymodel
Jul 8, 2026
Merged

Replace identitymodel#409
happymancoder merged 3 commits into
masterfrom
replace-identitymodel

Conversation

@MrSmoke

@MrSmoke MrSmoke commented Jul 3, 2026

Copy link
Copy Markdown
Member
  • Replaced the outdated IdentityModel with classes provided in Microsoft.IdentityModel.Protocols.OpenIdConnect
    • We can use ConfigurationManager as a replacement for the discovery cache as thats what the .AddJwtBearer() uses in aspnet
  • Added unit tests for ClickView.GoodStuff.AspNetCore.Authentication
  • Use the new JsonWebTokenHandler instead of the old JwtSecurityTokenHandler`
  • When checking the logout message, make sure we have Jti to match the spec

@MrSmoke
MrSmoke requested a review from happymancoder July 3, 2026 09:25
@MrSmoke MrSmoke changed the title Replace identitymodel WIP: Replace identitymodel Jul 3, 2026
@MrSmoke
MrSmoke force-pushed the replace-identitymodel branch from 8e7f5d6 to 1e67069 Compare July 3, 2026 10:05
@MrSmoke MrSmoke changed the title WIP: Replace identitymodel Replace identitymodel Jul 3, 2026
@happymancoder
happymancoder merged commit bf97beb into master Jul 8, 2026
4 of 5 checks passed
@happymancoder
happymancoder deleted the replace-identitymodel branch July 8, 2026 23:41
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