markmcdowell / NLog.Targets.ElasticSearch

NLog target for Elasticsearch
MIT License
176 stars 89 forks source link

Newtonsoft.Json dependency #139

Closed pirimoglu closed 3 years ago

pirimoglu commented 3 years ago

Newtonsoft.Json dependency can be changed with System.Text.Json

snakefoot commented 3 years ago

Yes but there are different side-effects:

See also: https://docs.microsoft.com/en-us/dotnet/standard/serialization/system-text-json-migrate-from-newtonsoft-how-to?pivots=dotnet-5-0

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

wonea commented 2 years ago

@snakefoot @pirimoglu Is it worth reassessing this question with .NET 6?

snakefoot commented 2 years ago

Guess one could look into trying to exercise the NEST-serializer, instead of using Newtonsoft.Json

wonea commented 2 years ago

On the roadmap for version 8.0 of elasticsearch-net

https://github.com/elastic/elasticsearch-net/issues/5941