Skip to content

update to Microsoft.IdentityModel.JsonWebTokens#370

Draft
omsmith wants to merge 1 commit into
mainfrom
omsmith/identitymodel-update2
Draft

update to Microsoft.IdentityModel.JsonWebTokens#370
omsmith wants to merge 1 commit into
mainfrom
omsmith/identitymodel-update2

Conversation

@omsmith

@omsmith omsmith commented Aug 1, 2024

Copy link
Copy Markdown
Contributor

Looks like a definite improvement. Need to spend a bit more time with it. Unit test is failing due to different claim order.

BenchmarkDotNet=v0.13.1, OS=Windows 10.0.22621
AMD Ryzen Threadripper PRO 7945WX 12-Cores, 1 CPU, 24 logical and 12 physical cores
  [Host]     : .NET Framework 4.8 (4.8.9256.0), X64 RyuJIT
  DefaultJob : .NET Framework 4.8 (4.8.9256.0), X64 RyuJIT


| Method |        Mean |    Error |   StdDev | Rank |
|------- |------------:|---------:|---------:|-----:|
|  ES256 | 1,216.47 us | 3.676 us | 3.259 us |    4 |
|  ES384 |   552.58 us | 3.321 us | 2.944 us |    3 |
|  ES512 |   255.73 us | 2.874 us | 2.548 us |    2 |
|  RS256 |    56.07 us | 0.444 us | 0.415 us |    1 |
BenchmarkDotNet=v0.13.1, OS=Windows 10.0.22621
AMD Ryzen Threadripper PRO 7945WX 12-Cores, 1 CPU, 24 logical and 12 physical cores
  [Host]     : .NET Framework 4.8 (4.8.9256.0), X64 RyuJIT
  DefaultJob : .NET Framework 4.8 (4.8.9256.0), X64 RyuJIT


| Method |        Mean |    Error |   StdDev | Rank |
|------- |------------:|---------:|---------:|-----:|
|  ES256 | 1,198.32 us | 6.115 us | 5.107 us |    4 |
|  ES384 |   540.46 us | 2.511 us | 2.348 us |    3 |
|  ES512 |   243.52 us | 1.124 us | 0.996 us |    2 |
|  RS256 |    41.21 us | 0.303 us | 0.283 us |    1 |

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.

1 participant