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

Request to backport JsonDocument thread safety fix to System.Text.Json v7 (PR#76716) #92808

Closed
jagdeepsingal opened this issue Sep 29, 2023 · 4 comments · Fixed by #92831 or #92832
Closed
Assignees
Milestone

Comments

@jagdeepsingal
Copy link

Hi,

This is to request for backport JsonDocument thread safety fix to System.Text.Json v7. This is the relevant PR: https://github.com/dotnet/runtime/pull/76716/files.

Thanks, and Best Regards,
Jagdeep Singal

@ghost ghost added the untriaged New issue has not been triaged by the area owner label Sep 29, 2023
@ghost
Copy link

ghost commented Sep 29, 2023

Tagging subscribers to this area: @dotnet/area-system-text-json, @gregsdennis
See info in area-owners.md if you want to be subscribed.

Issue Details

Hi,

This is to request for backport JsonDocument thread safety fix to System.Text.Json v7. This is the relevant PR: https://github.com/dotnet/runtime/pull/76716/files.

Thanks, and Best Regards,
Jagdeep Singal

Author: jagdeepsingal
Assignees: -
Labels:

area-System.Text.Json

Milestone: -

@danmoseley
Copy link
Member

@jagdeepsingal could you say more about why you want it ported? How is it affecting you?

@eiriktsarpalis
Copy link
Member

@jagdeepsingal System.Text.Json v8 is due to be released in November, which is probably sooner than the next servicing release that could accommodate such a backport. Using the latest STJ doesn't necessitate updating to the latest version of .NET, you can just consume the latest NuGet package. We've already released the RC1 version so you can try it out today.

@eiriktsarpalis eiriktsarpalis removed the untriaged New issue has not been triaged by the area owner label Sep 29, 2023
@eiriktsarpalis eiriktsarpalis added this to the 7.0.x milestone Sep 29, 2023
@eiriktsarpalis eiriktsarpalis self-assigned this Sep 29, 2023
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Sep 29, 2023
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Sep 30, 2023
@teo-tsirpanis
Copy link
Contributor

Closing, the issue has been backported to .NET 7 and 6.

@ghost ghost locked as resolved and limited conversation to collaborators Oct 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.