Azure / bicep

Bicep is a declarative language for describing and deploying Azure resources
MIT License
3.21k stars 745 forks source link

Switch to System.Text.Json #5886

Open stan-sz opened 2 years ago

stan-sz commented 2 years ago

Is your feature request related to a problem? Please describe. Switch to using System.Text.Json instead of JSON.Net.

See https://devblogs.microsoft.com/dotnet/try-the-new-system-text-json-apis/ and https://docs.microsoft.com/en-us/dotnet/standard/serialization/system-text-json-migrate-from-newtonsoft-how-to?pivots=dotnet-6-0

anthony-c-martin commented 2 years ago

We're using System.Text.Json for any new features we add, but note that the Deployment engine code is very tightly coupled to NewtonSoft - changes here would be extremely risky to existing users.

majastrz commented 2 years ago

For the reasons Anthony mentioned above, I don't think we'll be able to prioritize this work.

chunliu commented 2 years ago

Is it possible to at least migrate the decompiler from NewtonSoft to System.Text.Json? I'm using the decompiler in an Blazor WASM project. And using System.Text.Json is one of the performance best practices for Blazor WASM. https://docs.microsoft.com/en-us/aspnet/core/blazor/performance?view=aspnetcore-6.0#use-systemtextjson