microsoft / ApplicationInsights-Kubernetes

Enrich the telemetry data for .NET applications running inside containers that are managed by Kubernetes.
Other
138 stars 54 forks source link

Drop Newtonsoft.Json #307

Closed xiaomi7732 closed 2 years ago

xiaomi7732 commented 2 years ago

Drop explicit dependency to Newtonsoft.Json. By investigation, Newtonsoft.Json is still on the dependency chain. The intention is:

  1. No explicit dependency on it;
  2. When we are ready to upgrade to use a higher version of Kubernetes.Client, it won't be a blocking issue;

The bottom line, is there is nothing functionally wrong with using Newtonsoft.Json.