New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add System.Text.Json Support #356
Comments
That would be nice to have, but will be a very hard task since the library is highly coupled with newtonsoft.json, specifically with the use of the attributes So we'll need to first circunvent all the compile-time issues like this in order to provide a way to select the serialization mechanism at run-time. Feel free to make any suggestions for this |
I still think providing a mechanism to change the serialization at run-time will be very hard to accomplish. Maybe we can just default to use system.text.json internally intead of json.net on the assemblies targeting >= .net core 3, and if it was needed, provide alternative assemblies/packages to force the non-default serializarion mechanism (i.e. |
I've started trying with System.Text.Json and in less than 5 minutes I've found two open issues that will prevent a smooth migration:
So I think that, at this time, |
Added support to Refer to the following README sections: All the libraries were impacted since this was a very big change (184 files changed) so please expect some backward compatibility issues on the new serialization mechanism, even though all the unit tests and integration tests are passing, specially since the Please upgrade your references to version 18.0.0 and test. |
When targeting .NET Core 3.0 or higher using System.Text.Json will increase the performance and reduce the memory usage.
Maybe the json serializer could be like a provider that user can change it when configure Audit.NET
The text was updated successfully, but these errors were encountered: