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

[Breaking change]: System.Text.Json no longer hardcodes polymorphism for root-level object types #30758

Closed
1 of 2 tasks
eiriktsarpalis opened this issue Aug 19, 2022 · 0 comments · Fixed by #31132
Closed
1 of 2 tasks
Assignees
Labels
binary incompatible Existing binaries may encounter a breaking change in behavior. breaking-change Indicates a .NET Core breaking change 🏁 Release: .NET 7 Work items for the .NET 7 release doc-idea Indicates issues that are suggestions for new topics [org][type][category] Pri1 High priority, do before Pri2 and Pri3

Comments

@eiriktsarpalis
Copy link
Member

eiriktsarpalis commented Aug 19, 2022

Description

Using default configuration, System.Text.Json will serialize values of type object using polymorphism. This becomes less consistent once users register a custom converter for object, as witnessed in the following example:

using System;
using System.Text.Json;
using System.Text.Json.Serialization;

var options = new JsonSerializerOptions { Converters = { new CustomObjectConverter() } };

Console.WriteLine(JsonSerializer.Serialize<object>(0, options)); // Prints 0, custom converter not honored
Console.WriteLine(JsonSerializer.Serialize<object[]>(new object[] { 0 }, options)); // Prints [42], custom converter honored

public class CustomObjectConverter : JsonConverter<object>
{
    public override void Write(Utf8JsonWriter writer, object value, JsonSerializerOptions options)
        => writer.WriteNumberValue(42);

    public override object Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
        => throw new NotImplementedException();
}

In short, System.Text.Json has historically been hardcoding polymorphism for root-level object values but not using polymorphism for nested object values. Starting with .NET 7 RC1, this behavior has been changed so that custom converters never use polymorphism.

Version

.NET 7 RC 1

Previous behavior

Using the custom object converter defined above, the code

var options = new JsonSerializerOptions { Converters = { new CustomObjectConverter() } };
JsonSerializer.Serialize<object>(0, options);

Will serialize as 0, since the serializer will use polymorphism and ignore the custom converter.

New behavior

Using the custom object converter defined above, the code

var options = new JsonSerializerOptions { Converters = { new CustomObjectConverter() } };
JsonSerializer.Serialize<object>(0, options);

Will serialize as 42, since the serializer will always consult the custom converter and not use polymorphism.

Type of breaking change

  • Binary incompatible: Existing binaries may encounter a breaking change in behavior, such as failure to load/execute or different run-time behavior.
  • Source incompatible: Source code may encounter a breaking change in behavior when targeting the new runtime/component/SDK, such as compile errors or different run-time behavior.

Reason for change

Inconsistency on serialization contracts for a given type, depending on whether it is being serialized as a root-level value or a nested value.

Recommended action

Users can get back polymorphism for root-level values by invoking one of the untyped serialization methods:

var options = new JsonSerializerOptions { Converters = { new CustomObjectConverter() } };
JsonSerializer.Serialize(0, inputType: typeof(int), options); // serializes as 0

Feature area

Core .NET libraries

Affected APIs

No response

@eiriktsarpalis eiriktsarpalis added doc-idea Indicates issues that are suggestions for new topics [org][type][category] breaking-change Indicates a .NET Core breaking change Pri1 High priority, do before Pri2 and Pri3 labels Aug 19, 2022
@dotnet-bot dotnet-bot added ⌚ Not Triaged Not triaged 🏁 Release: .NET 7 Work items for the .NET 7 release binary incompatible Existing binaries may encounter a breaking change in behavior. labels Aug 19, 2022
@gewarren gewarren removed the ⌚ Not Triaged Not triaged label Aug 19, 2022
@ghost ghost added the in-pr This issue will be closed (fixed) by an active pull request. label Sep 13, 2022
@ghost ghost removed the in-pr This issue will be closed (fixed) by an active pull request. label Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binary incompatible Existing binaries may encounter a breaking change in behavior. breaking-change Indicates a .NET Core breaking change 🏁 Release: .NET 7 Work items for the .NET 7 release doc-idea Indicates issues that are suggestions for new topics [org][type][category] Pri1 High priority, do before Pri2 and Pri3
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants