search

Brightspace / D2L.Security.OAuth2

Brightspace OAuth 2.0 for C#
Apache License 2.0
7 stars 16 forks source link

update to Microsoft.IdentityModel.JsonWebTokens #370

Open omsmith opened 3 months ago

omsmith commented 3 months ago

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 |