Merge pull request #55 from 50c/dev

brunobritodev · web-flow · commit e03f3b219ce8 · 2024-04-18T21:20:05.000-03:00
close #47 close #54
diff --git a/src/NetDevPack.Security.Jwt.Core/Model/Key.cs b/src/NetDevPack.Security.Jwt.Core/Model/Key.cs
@@ -28,7 +28,10 @@ public KeyMaterial(CryptographicKey cryptographicKey)
 
     public JsonWebKey GetSecurityKey()
     {
-        return JsonSerializer.Deserialize<JsonWebKey>(Parameters);
+        //problem here are default Serializer options. If these options don't have the property PropertyNameCaseInsensitive = true, the deserialization will fail.
+        //var jsonWebKey = JsonSerializer.Deserialize<JsonWebKey>(Parameters);
+        var jsonWebKey = JsonWebKey.Create(Parameters);
+        return jsonWebKey;
     }
 
     public void Revoke(string reason=default)

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,10 @@ public KeyMaterial(CryptographicKey cryptographicKey)`
`28`	`28`
`29`	`29`	`public JsonWebKey GetSecurityKey()`
`30`	`30`	`{`
`31`		`- return JsonSerializer.Deserialize<JsonWebKey>(Parameters);`
	`31`	`+ //problem here are default Serializer options. If these options don't have the property PropertyNameCaseInsensitive = true, the deserialization will fail.`
	`32`	`+ //var jsonWebKey = JsonSerializer.Deserialize<JsonWebKey>(Parameters);`
	`33`	`+ var jsonWebKey = JsonWebKey.Create(Parameters);`
	`34`	`+ return jsonWebKey;`
`32`	`35`	`}`
`33`	`36`
`34`	`37`	`public void Revoke(string reason=default)`