You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As mentioned by @yufeih in #9717 (reply in thread), the use of NewtonSoft.Json in this library is due to the age of it. That's understandable.
However, using .NET's own System.Text.Json namespace should provide at least these obvious benefits:
Clear performance advantage.
One less third-party dependency.
I propose that a gradual move to System.Text.Json be made, despite it not being trivial. This move can coincide with the change to providing public config/options classes proposed in #9726.
Feel free to close if not planned or move to a "Backlog"-type milestone.
The text was updated successfully, but these errors were encountered:
Description
As mentioned by @yufeih in #9717 (reply in thread), the use of
NewtonSoft.Json
in this library is due to the age of it. That's understandable.However, using .NET's own
System.Text.Json
namespace should provide at least these obvious benefits:I propose that a gradual move to
System.Text.Json
be made, despite it not being trivial. This move can coincide with the change to providing public config/options classes proposed in #9726.Feel free to close if not planned or move to a "Backlog"-type milestone.
The text was updated successfully, but these errors were encountered: