Skip to content
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

Explain reason for using Newtonsoft.JSON #21229

Closed
mikewop opened this issue Jan 15, 2021 · 3 comments
Closed

Explain reason for using Newtonsoft.JSON #21229

mikewop opened this issue Jan 15, 2021 · 3 comments
Labels
needs-more-info Source - Docs.ms Docs Customer feedback via GitHub Issue

Comments

@mikewop
Copy link

mikewop commented Jan 15, 2021

With the switch to using System.Text.Json as the default in .Net 5, why is Newtonsoft.Json needed for JSON PATCH processing?
The webpage does not explain the rationale for having to use Newtonsoft.Json.

Is it a limitation in System.Text.Json? Is it being added/fixed in the future?


Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

@dotnet-bot dotnet-bot added ⌚ Not Triaged Source - Docs.ms Docs Customer feedback via GitHub Issue labels Jan 15, 2021
@Rick-Anderson
Copy link
Contributor

@serpent5 @tdykstra can we do this with System.Text.Json ?

@serpent5
Copy link
Contributor

Microsoft.AspNetCore.JsonPatch has used Newtonsoft.Json long before System.Text.Json was introduced. I don't think there are plans to update it to use System.Text.Json or to create a spin-out version.

See dotnet/aspnetcore#24333.

@Rick-Anderson
Copy link
Contributor

Thanks for contacting us. We believe that the question you've raised have been answered. If you still feel a need to continue the discussion, feel free to reopen it and add your comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-more-info Source - Docs.ms Docs Customer feedback via GitHub Issue
Projects
None yet
Development

No branches or pull requests

4 participants