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.

Sign up for GitHub

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

Jump to bottom

Fix deserializing mistmatched JToken types in properties #2530

Merged
merged 3 commits into from
May 14, 2021
Merged

Fix deserializing mistmatched JToken types in properties #2530

merged 3 commits into from
May 14, 2021

Conversation

JamesNK
Copy link
Owner

@JamesNK JamesNK commented May 14, 2021

Fixes #2529

@JamesNK JamesNK merged commit 2b0f7a2 into master May 14, 2021
@JamesNK JamesNK deleted the jamesnk/linqmistch-property-deserialize branch May 14, 2021 23:40
@ShaydeNofziger
Copy link

ShaydeNofziger commented Jan 4, 2022 •
edited
Loading

@JamesNK Hey James - thanks for all your hard work on this and the Newtonsoft.Json library!

Do you know if/when a new release version will be cut for this fix? Running into the issue now while updating from 12.0.3 to 13.0.1 and would prefer the fix vs the workarounds mentioned in #2529

@ejball ejball mentioned this pull request Mar 21, 2022
Open
DmitriiBobreshev pushed a commit to DmitriiBobreshev/azure-pipelines-agent that referenced this pull request Jul 5, 2022
Fix vulnerabilities for Newtonsoft.Json package by upgrading from 12 …
74f8755 
…to 13 beta version

-- Change version to beta because of the bug JamesNK/Newtonsoft.Json#2530
@ejball
Copy link

ejball commented Jul 5, 2022

@JamesNK Will this regression fix ever be deployed? It is especially critical now that older versions of the package are all alerted as potential security vulnerabilities.

@JamesNK
Copy link
Owner Author

JamesNK commented Jul 5, 2022

It's on NuGet

sousapedro pushed a commit to sousapedro/Newtonsoft.Json-for-Unity that referenced this pull request Feb 21, 2023
@JamesNK @sousapedro
Fix deserializing mistmatched JToken types in properties (JamesNK#2530)
4031f51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Exception deserializing JArray on JToken-typed property pre-initialized with JValue
3 participants
@JamesNK @ShaydeNofziger @ejball