azure-functions-extension
azure-functions-extension copied to clipboard
Use System.Text.Json instead of Newtonsoft
To avoid external dependencies like Newtonsoft, use System.Text.Json
I would recommend against this unless you have a really strong reason to remove this dependency. Some things to consider:
- All Functions SDKs already have this dependency already, whether they want it or not.
- The Functions runtime has deep integration with Newtonsoft.
- Adding System.Text.Json means that apps would have dependencies on two JSON SDKs instead of one.
@cgillum System.Text.Json would be used in this extension and its part of .net, AFAIK it doesn't need an extra nuget ref and apps would still be free to use S.T.J or Newtonsoft.
Would this extension add a transitive dependency on Newtonsoft through Functions nugets? Currently when users want to write a C# Function do they get Newtonsoft as a dependency from Functions? If the answer is yes, to both the above, then its fine to keep using Newtonsoft.